Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8737 )
Change subject: KUDU-1097. master: strip health reports from cstate before persisting ...................................................................... KUDU-1097. master: strip health reports from cstate before persisting The master should not persist configs with health reports in them because the health reports are inherently transient information. This patch strips the health reports from the cstate included in each leader tablet report before persisting them and adds a couple of DCHECKS in random places to enforce that. Change-Id: I3e0b076146d888d01676442f6ab3f67cca6bacd6 Reviewed-on: http://gerrit.cloudera.org:8080/8683 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Kudu Jenkins Reviewed-on: http://gerrit.cloudera.org:8080/8737 --- M src/kudu/master/catalog_manager.cc 1 file changed, 11 insertions(+), 1 deletion(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8737 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.6.x Gerrit-MessageType: merged Gerrit-Change-Id: I3e0b076146d888d01676442f6ab3f67cca6bacd6 Gerrit-Change-Number: 8737 Gerrit-PatchSet: 2 Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]>
