Hi Taher Alkhateeb,
wow, great work, thank you very much! Readability improved a lot!
I have read the code, run the tests and checked OFBiz works fine. I haven't checked the clustering functionality because I didn't have the time to setup a clustering environment to really test it. From the code, everything looks fine to me.
Thanks and regards,
Just one minor note: we currently set
context.setJ2EEServer("OFBiz Container 3.1");
I guess the "3.1" comes from ancient times and think we should remove it. It doesn't hurt because it seems only to be displayed in the JMX context but it looks strange.
+1 for committing it to the codebase.