Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19508 )
Change subject: IMPALA-11925: Added a new api add_write_notification_log_in_batch in the MetastoreServiceHandler class ...................................................................... Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/19508/4//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/19508/4//COMMIT_MSG@9 PS4, Line 9: In the MetastoreServiceHandler class, the This is now 42, the limit is 72. http://gerrit.cloudera.org:8080/#/c/19508/4//COMMIT_MSG@19 PS4, Line 19: 'add_write_notification_log_in_batch' api in HMS is reachable from impala. Line too long (74 vs 72). -- To view, visit http://gerrit.cloudera.org:8080/19508 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia9a35557c2ed79ed0276c4a418b5292fc6dd6194 Gerrit-Change-Number: 19508 Gerrit-PatchSet: 4 Gerrit-Owner: Sai Hemanth Gantasala <saihema...@cloudera.com> Gerrit-Reviewer: Aman Sinha <amsi...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: John Sherman <j...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Sai Hemanth Gantasala <saihema...@cloudera.com> Gerrit-Comment-Date: Fri, 10 Mar 2023 09:20:56 +0000 Gerrit-HasComments: Yes
