Consider refactoring the scope of synchronized blocks in AbstractCorrelatingMessageHandler [INT-2148]

[spring-operator]

Oleg Zhurakousky opened INT-2148 and commented

see discussion in this PR https://github.com/SpringSource/spring-integration/pull/78/files#r140610

---

Affects: 2.1 M1

This issue is a sub-task of #6135

Issue Links:

• Add locking strategy to AbstractMessageGroupStore [INT-1117] #5123 Add locking strategy to AbstractMessageGroupStore
  ("is superseded by")

[spring-operator]

Oleg Zhurakousky commented

Looking at it further I think the current scope of the synchronized blocks is good where it is. Having said that all synchronized blocks are useless in the distributed environment, but until we address the #5123 we should keep them where they are.

[spring-operator]

Mark Fisher commented

I'm resolving this, but I first linked it as "superseded by" #5123.