Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17152 )
Change subject: KUDU-3254 fix bug in meta-cache exposed by KUDU-1802 ...................................................................... Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/17152/6/src/kudu/client/meta_cache.cc File src/kudu/client/meta_cache.cc: http://gerrit.cloudera.org:8080/#/c/17152/6/src/kudu/client/meta_cache.cc@932 PS6, Line 932: if (remote.get() != nullptr) { > Yep: in pre-KUDU-1802 case, the crash could only happen if the range partit A small update: in pre-KUDU-1802, a crash could only happen if a range was dropped and replaced with the same range again _and_ the UUID of the new tablet is the same as it was. The latter is virtually impossible because UUID would not be the same. So, in pre-KUDU-1802 case this would not trigger. -- To view, visit http://gerrit.cloudera.org:8080/17152 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.13.x Gerrit-MessageType: comment Gerrit-Change-Id: I5b8370290c13b1e496f461ed5bc2e0193bdf4b19 Gerrit-Change-Number: 17152 Gerrit-PatchSet: 7 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Sun, 07 Mar 2021 15:45:23 +0000 Gerrit-HasComments: Yes
