Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15971 )

Change subject: WIP [master] cache for table locations
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15971/1//COMMIT_MSG
Commit Message:

PS1:
I also did some integration-level testing with a synthetic workload against a 
real Kudu cluster.   It turned out that the performance improvement is 
negligible when turning on the cache for tablet locations.

Maybe, there are some specific partition schemas that could benefit more, but 
at this point I'm not sure it's worth caching the locations, at least as it's 
done in this patch.



--
To view, visit http://gerrit.cloudera.org:8080/15971
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7d2a4771ddc455d92a1da00db91c555a21151a23
Gerrit-Change-Number: 15971
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 21 May 2020 21:51:33 +0000
Gerrit-HasComments: Yes

Reply via email to