Michael Smith has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/19643 )

Change subject: IMPALA-12014 warning on failed KeepAlive for Kudu scanner
......................................................................


Patch Set 1:

(2 comments)

This seems

http://gerrit.cloudera.org:8080/#/c/19643/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/19643/1//COMMIT_MSG@15
PS1, Line 15: This is a follow-up to f5e8047886ae7c7a9466319d86ddf9bb0c7801c3.
nit: It's more searchable to reference the ticket number rather than commit.


http://gerrit.cloudera.org:8080/#/c/19643/1/be/src/exec/kudu/kudu-scanner.cc
File be/src/exec/kudu/kudu-scanner.cc:

http://gerrit.cloudera.org:8080/#/c/19643/1/be/src/exec/kudu/kudu-scanner.cc@111
PS1, Line 111:     LOG(WARNING) << BuildErrorString(
Could this generate lots of spurious warnings? Is failing to deliver a 
keepalive a sign of overloaded replicas, or something that happens pretty often 
generally?



--
To view, visit http://gerrit.cloudera.org:8080/19643
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If39f968685797506491d3d5ebc481efb6e43a568
Gerrit-Change-Number: 19643
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Comment-Date: Tue, 21 Mar 2023 20:52:48 +0000
Gerrit-HasComments: Yes

Reply via email to