-
1. Synchronized method
wdfink Jan 26, 2011 2:27 AM (in response to born_free_77)Yes, yes
synchronized is against the JEE specification
and
in my experience (I found such code in different applications and wrote it as beginner ;-) ) such code harm the throughput and during the time you spend a lot of time to analyze fuzzy behaviour regarding this.
A better approach will be to explain what you want and as for un-synchronized solution.
BTW, if you try to install a cluster the sync is gone because of the different JVMs ;-)
-
2. Synchronized method
born_free_77 Jan 26, 2011 9:56 AM (in response to wdfink)Thanks for the response,
This code is the existing code in an application which I was working. The app is deployed and it works fine but recently we have an issue with one of our batch process and some fail but most of them pass. My suspicion is this method which is marked as synchronized.
I was trying to find a test case to prove that synch method do harm, if any one know the test case pls let me know.
Thanks,
-
3. Synchronized method
born_free_77 Jan 26, 2011 12:16 PM (in response to born_free_77)Som more info on the above ejb code.
MyWorkflow1 and MyWorkflow2 are defined as ejb using xdoclet.
MySuperClass extends SessionBean
someEJBMethod() is defined as ejb.interface method using xdoclet
aMehod() defined in super class is a public method and has no xdoclet defined.
-
4. Synchronized method
wdfink Jan 26, 2011 1:37 PM (in response to born_free_77)That is what I mean, unexpected behaviour under load.
I'm sure that your code work under 'normal' conditions, but this is one of the reasons why you do not use synchronized for EJBs.
What would help but falsified the result is a profiler (like jProfiler), but for such behaviour it is useless.
We have a good experience by using thread dumps and compare 2 or more dumps (e.g. 10sec gap) looking for blocked threads, you will find the blocking object via ID and you will see a number of threads waiting for this.
A general test case is not possible, you might stress this use case in a test environment and increase the parallel access, e.g. start with 1 and add 5 each minute. The result is a load profile which does not scale after a threshold.
hope this will be helpful
-
5. Synchronized method
born_free_77 Feb 1, 2011 4:59 PM (in response to wdfink)I wrote a test case to test this but was not able to see any undesired result.
I created 2 beans each extending abstract bean class which has synchronized method. Each bean calls that method by calling super.aMethod();
My controller class creates a custom class which is then submitted to a Executor service class which has thread pool defined.
I initially had only 20 threads in the pool and later I increased it.