You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Yingchun Lai (Code Review)" <ge...@cloudera.org> on 2022/05/16 23:56:04 UTC

[kudu-CR] [tools] update schema if needed when rebuilding master

Hello Alexey Serbin, Kudu Jenkins, Abhishek Chennaka, Andrew Wong, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/18496

to look at the new patch set (#6).

Change subject: [tools] update schema if needed when rebuilding master
......................................................................

[tools] update schema if needed when rebuilding master

Before this patch, if tservers had outdated schemas with lower
version for some tables (e.g. when the cluster wasn't healthy), the
'kudu master unsafe_rebuild' tool might rebuild the system catalog
with outdated schemas for the unhealthy tables.

This patch optimizes the rebuild logic, so the tool now picks up
schema of the highest version for an unhealthy table when rebuilding
the system catalog.

Change-Id: Iec99d57115228b521ba645b8e19c7057a4bb5d3d
---
M src/kudu/tools/kudu-admin-test.cc
M src/kudu/tools/master_rebuilder.cc
2 files changed, 125 insertions(+), 44 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/96/18496/6
-- 
To view, visit http://gerrit.cloudera.org:8080/18496
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iec99d57115228b521ba645b8e19c7057a4bb5d3d
Gerrit-Change-Number: 18496
Gerrit-PatchSet: 6
Gerrit-Owner: Yingchun Lai <ac...@gmail.com>
Gerrit-Reviewer: Abhishek Chennaka <ac...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <ac...@gmail.com>