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

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


Patch Set 14:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/20282/12/src/kudu/client/client.h
File src/kudu/client/client.h:

http://gerrit.cloudera.org:8080/#/c/20282/12/src/kudu/client/client.h@2962
PS12, Line 2962:   /// Stop keeping the current remote scanner alive 
periodically.
> In which case we need to stop the timer manually?
For example, there are many tables to be read and many scanners be created to 
read tables. If reading one of tables failed, the program is designed to skip 
the failed tables and continue reading other tables. It is better to stop the 
keep-alive timer. Besides, for some unit tests, it needs this interface.



--
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: 14
Gerrit-Owner: Wang Xixu <1450306...@qq.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <mre...@cloudera.com>
Gerrit-Reviewer: Wang Xixu <1450306...@qq.com>
Gerrit-Reviewer: Yifan Zhang <chinazhangyi...@163.com>
Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org>
Gerrit-Reviewer: Yuqi Du <shenxingwuy...@gmail.com>
Gerrit-Comment-Date: Wed, 11 Oct 2023 03:26:04 +0000
Gerrit-HasComments: Yes

Reply via email to