Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18866 )
Change subject: [master] KUDU-3322 / KUDU-3319 HMS event logging ...................................................................... [master] KUDU-3322 / KUDU-3319 HMS event logging It would be useful to have additional logging in cases where HMS notification events have to be analyzed from Kudu side. The Jiras have more context on the reasoning behind this but this patch adds * an INFO level log which shows the last processed HMS event id during startup and * an ERROR log if the latest event id from HMS is less than the last seen event id from the master. Was hesitant on crashing the master and chose to log an ERROR as in the case an out of order event id received from HMS might lead to unnecessary crash of the Kudu master. Change-Id: I242c9cedf170ed621867b905f1cb1875347d5887 Reviewed-on: http://gerrit.cloudera.org:8080/18866 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/hms/hms_catalog.cc M src/kudu/hms/hms_catalog.h M src/kudu/hms/mini_hms.cc M src/kudu/hms/mini_hms.h M src/kudu/integration-tests/master_hms-itest.cc M src/kudu/master/catalog_manager.cc M src/kudu/master/hms_notification_log_listener.cc 7 files changed, 82 insertions(+), 3 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/18866 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I242c9cedf170ed621867b905f1cb1875347d5887 Gerrit-Change-Number: 18866 Gerrit-PatchSet: 8 Gerrit-Owner: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
