Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15971 )
Change subject: [master] cache for table locations ...................................................................... Patch Set 7: (1 comment) > Build Failed > > http://jenkins.kudu.apache.org/job/kudu-gerrit/21904/ : FAILURE It seems the tests need re-work. http://gerrit.cloudera.org:8080/#/c/15971/2/src/kudu/master/catalog_manager.cc File src/kudu/master/catalog_manager.cc: http://gerrit.cloudera.org:8080/#/c/15971/2/src/kudu/master/catalog_manager.cc@5130 PS2, Line 5130: e->GetTabletsInRange(r > Hmm.. Do you think it's worth predicating on FLAGS_table_locations_cache_ca Not sure how it's relevant: if master is not a leader, this -- 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: 7 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Bankim Bhavsar <ban...@cloudera.com> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Comment-Date: Tue, 30 Jun 2020 17:37:42 +0000 Gerrit-HasComments: Yes