Ashwani Raina has posted comments on this change. ( http://gerrit.cloudera.org:8080/19653 )
Change subject: [client] Improve logging in client metacache ...................................................................... Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/19653/5//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/19653/5//COMMIT_MSG@9 PS5, Line 9: As part for impala crashing issue > Adding this impala's jira issue at this commit message would help us to und I have mentioned the jira (KUDU-3461) in previous comments. However, it would make sense to add the jira id here for posterity. Done. http://gerrit.cloudera.org:8080/#/c/19653/5/src/kudu/client/meta_cache.cc File src/kudu/client/meta_cache.cc: http://gerrit.cloudera.org:8080/#/c/19653/5/src/kudu/client/meta_cache.cc@465 PS5, Line 465: if (leader) { : VLOG(1) << "Client copy of tablet " << tablet_->tablet_id() : << " is fresh, leader uuid " << leader->permanent_uuid(); : } else { : VLOG(1) << "Client copy of tablet " << tablet_->tablet_id() : << " is fresh (no leader)."; : } > nit: I don't mind using that. But I am not sure whether there is there any advantage (performance wise or other) of using 'Substitute' over standard usage. -- To view, visit http://gerrit.cloudera.org:8080/19653 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2a47f643307ef47a98b1d3481b4c03834fa239d4 Gerrit-Change-Number: 19653 Gerrit-PatchSet: 5 Gerrit-Owner: Ashwani Raina <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Ashwani Raina <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Yuqi Du <[email protected]> Gerrit-Comment-Date: Fri, 31 Mar 2023 07:53:00 +0000 Gerrit-HasComments: Yes
