Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11877 )
Change subject: Add TS UUID to alter_table-randomized-test "restart TS" step ...................................................................... Add TS UUID to alter_table-randomized-test "restart TS" step While debugging a failure of the randomized alter test, I needed to know which tablet server was restarted. The test's logs for it are I1102 21:39:02.323911 7768 alter_table-randomized-test.cc:120] Restarting TS 2 I1102 21:39:02.430003 7768 alter_table-randomized-test.cc:125] TS 2 Restarted which don't help me so much because I don't know which UUID is which index. It's possible to use the startup messages in the logs to figure out which UUID was restarted, but it'd be easier if it were just logged. So that's what it does now: I1105 10:17:26.197420 2830984064 alter_table-randomized-test.cc:122] Restarting TS fef46d70bd71410c8caa0aeb2a3cbafc (index 2) I1105 10:17:26.375321 2830984064 alter_table-randomized-test.cc:126] TS fef46d70bd71410c8caa0aeb2a3cbafc (index 2) restarted Change-Id: Ifbd2ee5ca3e3d7788812162e6188905adc3a6ec3 Reviewed-on: http://gerrit.cloudera.org:8080/11877 Reviewed-by: Andrew Wong <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/integration-tests/alter_table-randomized-test.cc 1 file changed, 8 insertions(+), 7 deletions(-) Approvals: Andrew Wong: Looks good to me, approved Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11877 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ifbd2ee5ca3e3d7788812162e6188905adc3a6ec3 Gerrit-Change-Number: 11877 Gerrit-PatchSet: 2 Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Will Berkeley <[email protected]>
