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

Guozhang Wang commented on KAFKA-6412:
--------------------------------------

[~tedyu] way to start 2018 indeed :)

> Improve synchronization in CachingKeyValueStore methods
> -------------------------------------------------------
>
>                 Key: KAFKA-6412
>                 URL: https://issues.apache.org/jira/browse/KAFKA-6412
>             Project: Kafka
>          Issue Type: Improvement
>          Components: streams
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>             Fix For: 1.1.0
>
>         Attachments: 6412-jmh.v1.txt, k-6412.v1.txt
>
>
> Currently CachingKeyValueStore methods are synchronized at method level.
> It seems we can use read lock for getter and write lock for put / delete 
> methods.
> For getInternal(), if the underlying thread is streamThread, the 
> getInternal() may trigger eviction. This can be handled by obtaining write 
> lock at the beginning of the method for streamThread.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to