[JBoss JIRA] Created: (JBAS-5477) Ejb3Deployer does not create DeploymentScope for DeploymentUnit with no parent
by Andrew Lee Rubinger (JIRA)
Ejb3Deployer does not create DeploymentScope for DeploymentUnit with no parent
------------------------------------------------------------------------------
Key: JBAS-5477
URL: http://jira.jboss.com/jira/browse/JBAS-5477
Project: JBoss Application Server
Issue Type: Bug
Security Level: Public (Everyone can see)
Components: EJB3
Reporter: Andrew Lee Rubinger
Assigned To: Scott M Stark
Priority: Blocker
Fix For: JBossAS-5.0.0.CR1
Revision 72296:
http://fisheye.jboss.org/browse/JBossAS/trunk/ejb3/src/main/org/jboss/ejb...
...has made the change that a null scope is passed into the ctor of Ejb3JBoss5Deployment, causing an NPE later down the chain:
Caused by: java.lang.NullPointerException
at org.jboss.ejb3.deployers.JBoss5DeploymentScope.getEjbContainer(JBoss5DeploymentScope.java:155)
at org.jboss.ejb3.Ejb3Deployment.getEjbContainer(Ejb3Deployment.java:388)
at org.jboss.ejb3.EJBContainer.resolveEjbContainer(EJBContainer.java:1320)
...etc
I'd tried setting:
scope = new JBoss5DeploymentScope(unit);
...in the case that parent==null, but this lead to deployment errors of other services on Startup. Additionally,
unit.getAttachment(DeploymentScope.class)
...returns null.
To test, run the "service" integration tests from the EJB3 TestSuite.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
18 years, 3 months
[JBoss JIRA] Created: (JGRP-736) GMS.ViewHandler.run() view budling not correctly checking if requests can be processed together...
by Michael Newcomb (JIRA)
GMS.ViewHandler.run() view budling not correctly checking if requests can be processed together...
--------------------------------------------------------------------------------------------------
Key: JGRP-736
URL: http://jira.jboss.com/jira/browse/JGRP-736
Project: JGroups
Issue Type: Bug
Reporter: Michael Newcomb
Assigned To: Bela Ban
Consider:
1356 do {
1357 Request firstRequest=(Request)q.remove(INTERVAL); // throws a TimeoutException if it runs into timeout
1358 requests.add(firstRequest);
1359 if(q.size() > 0) {
1360 Request nextReq=(Request)q.peek();
1361 keepGoing=view_bundling && firstRequest.canBeProcessedTogether(nextReq);
1362 }
1363 else {
1364 stop=System.currentTimeMillis();
1365 wait_time=max_bundling_time - (stop-start);
1366 if(wait_time > 0)
1367 Util.sleep(wait_time);
1368 keepGoing=q.size() > 0;
1369 }
1370 }
1371 while(keepGoing);
The faulty logic occurs on line 1368. At this point there was only 1 object on the queue when it was removed, so it waited max_bundling_time. After the wait_time expires, it checks to see if q.size() > 0, if so, it keeps going.
This actually creates 2 problems.
1. if view_bundling is false, it will still bundle the next request because it goes right back up and removes it and adds it to requests
2. if the request in the queue can't be processed with the request that just came in, it will still be added to requests
So, the simple solution (for released versions) is to use the checks from 1360-1361:
1367.5 Request nextReq=(Request)q.peek();
1368 keepGoing=view_bundling && ((Request) q.getLast()).canBeProcessedTogether(nextReq);
I think the ViewHandler could be rewritten for later releases so that as requests come in, it notifies the ViewHandler thread and he can check all queued requests to see if they could be processed together. Therefore, if you had view_bundling enabled, but a request comes in that can't be processed with existing requests, it will start processing immediately instead of waiting the rest of max_bundling_time (not long, but could be if configured that way).
Also, if view_bundling is disabled, it should not sleep at all.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
18 years, 3 months