Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20282 )

Change subject: KUDU-3498 Scanner keeps alive in periodically
......................................................................


Patch Set 25:

> (3 comments)

It seems the newly added tests are flaky, and they fail a lot since this patch 
has been pushed into the central git repo:

  http://dist-test.cloudera.org:8080/test_drilldown?test_name=client-test

That's not good since there might be more unrelated test failures of pre-commit 
tests for upcoming patches.

In addition, there is another issue with the new code as reported in 
https://issues.apache.org/jira/browse/KUDU-3524

@Wang Xixu, when you have a chance, could you please take care of the flakiness 
and KUDU-3524?  If you don't think you are going to have a chance working on 
that in the nearest future, please let me know, so I could allocate some time 
for fixing that.

Thank you!


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1165d96814eb4bcd5db9b5cb60403fffc5b18c81
Gerrit-Change-Number: 20282
Gerrit-PatchSet: 25
Gerrit-Owner: Wang Xixu <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Wang Xixu <[email protected]>
Gerrit-Reviewer: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Yuqi Du <[email protected]>
Gerrit-Comment-Date: Tue, 14 Nov 2023 04:08:08 +0000
Gerrit-HasComments: No

Reply via email to