Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13205 )
Change subject: [authz] add scrubbing task for the privileges cache ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/13205/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/13205/1//COMMIT_MSG@10 PS1, Line 10: The : period of the scrubbing thread is a half of an entry's TTL. > How did you arrive at this value? Updated -- I decided to go with 20 seconds by default. The idea is that the TTL for an entry in privileges cache might be relatively long, so it doesn't make sense to make it dependent on that parameter. -- To view, visit http://gerrit.cloudera.org:8080/13205 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I88510fae48cf683fbfad8a11c2941a23b2af9f8b Gerrit-Change-Number: 13205 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 01 May 2019 08:01:51 +0000 Gerrit-HasComments: Yes
