Abhishek Chennaka has uploaded this change for review. ( 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 --- 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, 67 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/66/18866/1 -- 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: newchange Gerrit-Change-Id: I242c9cedf170ed621867b905f1cb1875347d5887 Gerrit-Change-Number: 18866 Gerrit-PatchSet: 1 Gerrit-Owner: Abhishek Chennaka <[email protected]>
