Cargo
  1. Cargo
  2. CARGO-859

JBoss remote deployer doesn't work with a large war file

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.3
    • Fix Version/s: 1.0.4
    • Component/s: JBoss, Maven2
    • Labels:
      None
    • Complexity:
      Intermediate
    • Tested on JDKs:
      1.5.15
    • Number of attachments :
      0

      Description

      I have trouble deploying a large war file to a remote JBoss server using the cargo-maven2 plugin from an other server.

      This feature work's fine with a small war file (~ 5kb), but when i use it with a real life war file (~10 mb), nothing happens after more than 40 minutes, perhaps a concurrency problem !

      I think that the problem come from the implementation of the embdeded Web server (included in the 1.0.3 of Cargo to allow uploading a file to the remote server).

      My collegue Cedric Ghanassia has submited a patch that implements the upload feature to a remote JBoss application server using an embeded Jetty Server.

      I think that's a good idea to include a robust http server like Jetty, because implementing a http server from scratch is not very easy.

      Kind regards

      Boubakar

        Issue Links

          Activity

          Hide
          Savas Ali Tokmen added a comment -

          I guess you're using JBoss 4... Which version exactly?

          Apart from that, we have already discussed in CARGO-416 that embedding ANY version of Jetty will NEVER be done in any version of CARGO.

          Show
          Savas Ali Tokmen added a comment - I guess you're using JBoss 4... Which version exactly? Apart from that, we have already discussed in CARGO-416 that embedding ANY version of Jetty will NEVER be done in any version of CARGO.
          Hide
          Savas Ali Tokmen added a comment -

          The current implementation of the simple HTTP server doesn't actually provide any "Content-Length" header. I suspect JBoss' HTTP client cannot read large amounts of data if it is not told exactly how much to read.

          With revision 2528, I have added that missing HTTP header. It does not break any existing tests, please try out that version with your big WAR.

          If you cannot compile CARGO, I can also redeploy the SNAPSHOTs.

          Thank you

          Show
          Savas Ali Tokmen added a comment - The current implementation of the simple HTTP server doesn't actually provide any "Content-Length" header. I suspect JBoss' HTTP client cannot read large amounts of data if it is not told exactly how much to read. With revision 2528, I have added that missing HTTP header. It does not break any existing tests, please try out that version with your big WAR. If you cannot compile CARGO, I can also redeploy the SNAPSHOTs. Thank you
          Hide
          Savas Ali Tokmen added a comment -

          Just uploaded the new version to SNAPSHOT.

          Instructions on: http://cargo.codehaus.org/Maven2+Plugin+Installation

          Show
          Savas Ali Tokmen added a comment - Just uploaded the new version to SNAPSHOT. Instructions on: http://cargo.codehaus.org/Maven2+Plugin+Installation
          Hide
          Savas Ali Tokmen added a comment -

          Just tried on my side, I confirm that it is NOT working.

          Show
          Savas Ali Tokmen added a comment - Just tried on my side, I confirm that it is NOT working.
          Hide
          Boubakar Saim haddache added a comment -

          Thank you for your quick answer.

          We're using the JBOSS 4.3.0

          Show
          Boubakar Saim haddache added a comment - Thank you for your quick answer. We're using the JBOSS 4.3.0
          Hide
          Savas Ali Tokmen added a comment -

          Great.

          On my side I've just made it work. Can you please try SVN revision 2529?

          Thanks

          Show
          Savas Ali Tokmen added a comment - Great. On my side I've just made it work. Can you please try SVN revision 2529? Thanks
          Hide
          Savas Ali Tokmen added a comment -

          Just uploaded the new SNAPSHOT version (revision 2529).

          Instructions on: http://cargo.codehaus.org/Maven2+Plugin+Installation

          Once this issue is resolved, we'll be starting the CARGO 1.0.4 release.

          Show
          Savas Ali Tokmen added a comment - Just uploaded the new SNAPSHOT version (revision 2529). Instructions on: http://cargo.codehaus.org/Maven2+Plugin+Installation Once this issue is resolved, we'll be starting the CARGO 1.0.4 release.
          Hide
          Boubakar Saim haddache added a comment -

          Thank you Savas Ali.

          Show
          Boubakar Saim haddache added a comment - Thank you Savas Ali.
          Hide
          Savas Ali Tokmen added a comment -

          Is it working, then?

          Show
          Savas Ali Tokmen added a comment - Is it working, then?
          Hide
          Boubakar Saim haddache added a comment -

          Thank you, now it works.

          Thank you

          Show
          Boubakar Saim haddache added a comment - Thank you, now it works. Thank you

            People

            • Assignee:
              Savas Ali Tokmen
              Reporter:
              Boubakar Saim haddache
            • Votes:
              1 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: