Cargo

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

is depended upon by

Bug - A problem which impairs or prevents the functions of the product. CARGO-511 Deployment to JBoss4x not reliable

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
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
Permalink
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
Permalink
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
Permalink
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
Permalink
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
Permalink
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
Permalink
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
Permalink
Boubakar Saim haddache added a comment -

Thank you Savas Ali.

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

Is it working, then?

Show
Savas Ali Tokmen added a comment - Is it working, then?
Hide
Permalink
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
Vote (1)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: