[
https://issues.apache.org/jira/browse/GEODE-7863?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17072049#comment-17072049
]
ASF subversion and git services commented on GEODE-7863:
--------------------------------------------------------
Commit 4ede81abb2393bddbfb0e297cfc7bdfa3fee800b in geode's branch
refs/heads/master from Juan José Ramos
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=4ede81a ]
GEODE-7863: Reduce ServerCQImpl Contention (#4798)
We don't need to lock the entire internal cache for Partitioned
regions so the implementation is now split by region type, this will
us to improve/change them independently in the future.
- Removed redundant checks.
- Keep current behavior for Replicate Regions.
- Use ConcurrentMap instead of locking the entire internal cache on
every operation for Partition Regions.
- Keep the lock on ServerCQImpl instance only while executing the
query and leave stats operations outside of the synchronized block.
(cherry picked from commit 9e3c473e9cf6d72126d0e84319b1192a5e2b6fdb)
> Reduce Contention on ServerCQImpl
> ---------------------------------
>
> Key: GEODE-7863
> URL: https://issues.apache.org/jira/browse/GEODE-7863
> Project: Geode
> Issue Type: Bug
> Components: cq
> Reporter: Juan Ramos
> Assignee: Juan Ramos
> Priority: Major
> Labels: GeodeCommons
> Fix For: 1.13.0
>
> Time Spent: 2h 10m
> Remaining Estimate: 0h
>
> While looking at some performance issues recently introduced into {{Geode}}
> I've noticed that the several methods from the {{ServerCQImpl}} class
> internally lock an internal {{HashMap}}, it would be better to use a
> {{ConcurrentMap}} instead.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
