Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11460 )
Change subject: [hms] relax the notification format restriction ...................................................................... [hms] relax the notification format restriction 'messageFormat' is an optional field in NotificationEvent thrift object introduced in HIVE-10562. In order to be compatible with Hive distributions that do not include HIVE-10652, this commit relaxes the restriction of HMS notification message format, so that the notification log listener can process the message even when the format type is not provided. Removing this restriction should not cause any regressions as long as the required fields are present in the messages. Change-Id: I09d0e36cebe2074d975504d6bbc5677107e33fe3 Reviewed-on: http://gerrit.cloudera.org:8080/11460 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/master/hms_notification_log_listener.cc 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/11460 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I09d0e36cebe2074d975504d6bbc5677107e33fe3 Gerrit-Change-Number: 11460 Gerrit-PatchSet: 4 Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins
