History | Log In     View a printable version of the current page. Get help!  
Issue Details (XML | Word)

Key: JBPAPP-360
Type: Task Task
Status: Resolved Resolved
Resolution: Done
Priority: Minor Minor
Assignee: Brian Stansberry
Reporter: Brian Stansberry
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
JBoss Enterprise Platform App Edition

Re-enable JGroups message bundling

Created: 18/Oct/07 04:32 PM   Updated: 18/Dec/07 04:13 PM
Component/s: App Server
Affects Version/s: None
Fix Version/s: 4.2.0.GA_CP02 , 4.3.0.BETA3
Security Level: Public (Everyone can see)

Original Estimate: Unknown Remaining Estimate: Unknown Time Spent: Unknown
Issue Links:
Dependency
This issue depends:
JBPAPP-359 Upgrade to JGroups 2.4.1.SP4 Major Resolved
 

Affects: Release Notes


 Description  « Hide
For 4.2.0 JGroups message bundling was disabled pending investigation of JGRP-497. That is fixed in JGroups 2.4.1.SP4. Message bundling improves performance, so we should re-enable it.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order:
Rajesh Rajasekaran [07/Nov/07 11:34 AM]
Dominik, this issue should be fixed with the EAP4.3 beta build which includes JGroups 2.4.1.SP4
Please verify the issue and close it.

Brian Stansberry [07/Nov/07 11:51 AM]
This is not fixed, unless someone changed config files without telling me.

Rajesh Rajasekaran [07/Nov/07 11:55 AM]
Brian, the config files are not changed. Can you take a look at it?
I am glad atleast this was brought to your attention now.
The jgroups upgrade is already in place in branch JBPAPP_4_2.
If you can change the config files, it will automatically be picked up in the next build for testing.

Brian Stansberry [07/Nov/07 02:54 PM]
Config files are changed on JBPAPP_4_2 to re-enable bundling. I'm resolving for 4.2.0.GA_CP02 as well under the assumption this will get onto the CP branch as part of the merge.

Assigning to Dominik per Rajesh's earlier comment.

Dominik Pospisil [08/Nov/07 08:50 AM]
In 4.3.0 Beta1, message bundling is still disabled for all services relying on JGroups.

Rajesh Rajasekaran [08/Nov/07 08:57 AM]
Dominik, Brian committed the config changes yesterday which is not available in the beta1 build. Please have this issue to be verified on the next build.

Brian Stansberry [13/Nov/07 04:35 PM]
Assigning to Dominik to verify.

Dominik Pospisil [20/Nov/07 03:43 AM]
Verified that bundling is enabled for all services except messaging. Is this intended?

*/deploy/jboss-messaging.sar/hsqldb-persistence-service.xml still has bundling disabled for both data and control channels.

Brian Stansberry [20/Nov/07 08:45 AM]
I'll check with Tim Fox; that's up to him.

Brian Stansberry [21/Nov/07 12:13 PM]
I am going to return enable_bundling to 'false' on the cluster-service.xml stack:

1) This channel is not expected to have high concurrent load. Enabling bundling helps performance under high concurrent load but adds latency under low load.
2) Bundling was disabled in 4.0.5.GA; disabling it in 4.2 was not a "takeaway", so less reason to enable it now.
3) The added latency causes StatefulPassivationActivationUnitTestCase to fail with a timeout. That's just due to how the test is designed; the small bit of latency happens to get triggered thousands of times by a single thread.

#3 isn't a good enough reason to disable bundling, but it helps concentrate the mind on issue #1.