You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by "Ted Yu (JIRA)" <ji...@apache.org> on 2017/12/30 08:20:00 UTC

[jira] [Commented] (KAFKA-6412) Improve synchronization in CachingKeyValueStore methods

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

Ted Yu commented on KAFKA-6412:
-------------------------------

[~damianguy] [~guozhang] [~mjsax] :
See if I missed something.

> Improve synchronization in CachingKeyValueStore methods
> -------------------------------------------------------
>
>                 Key: KAFKA-6412
>                 URL: https://issues.apache.org/jira/browse/KAFKA-6412
>             Project: Kafka
>          Issue Type: Improvement
>            Reporter: Ted Yu
>
> 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)