[ 
https://issues.apache.org/jira/browse/OFBIZ-13153?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17890340#comment-17890340
 ] 

Jacques Le Roux commented on OFBIZ-13153:
-----------------------------------------

Hi Mayank ,

As synchronizng a whole method can have side effects, I'd suggest to start a 
thread in [dev ML|https://ofbiz.apache.org/mailing-lists.html] before blindly 
applying this change to OOTB code, TIA

> "null DispatchContext" issue under high load and concurrency
> ------------------------------------------------------------
>
>                 Key: OFBIZ-13153
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-13153
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework/service
>    Affects Versions: 18.12.16
>         Environment: Virtual Machine:
> OS: RHEL 8.9
> VCPU: 16
> Total RAM: 64 GB
> Heap allocation (Xmx): 8 GB
>            Reporter: Mayank S
>            Priority: Major
>              Labels: performance
>             Fix For: 18.12.17
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> While performance-testing Ofbiz 18 under high load (200 concurrent users), we 
> have observed an intermittent error, stating "null DispatchContext".
> While doing more analysis, we realized it is a thread-safety issue, and it 
> was fixed by adding the 'synchronized' keyword to the following method 
> signatures in ServiceDispatcher.java:
> public static synchronized getInstance (Delegator delegator) 
> public static synchronized void register (DispatchContext context)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to