YangSong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14793 )

Change subject: [test] Avoid mod 0, resolve an exception like 'Floating point 
exception'.
......................................................................


Patch Set 3:

(2 comments)

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

http://gerrit.cloudera.org:8080/#/c/14793/1//COMMIT_MSG@7
PS1, Line 7: [test] Avoid mod 0, resolve an exception like 'Floating point 
exception'.
> Nit: This subject is too long. Can you set a brief subject and then use the
I try to modify it.


http://gerrit.cloudera.org:8080/#/c/14793/1/src/kudu/tserver/tablet_server-test.cc
File src/kudu/tserver/tablet_server-test.cc:

http://gerrit.cloudera.org:8080/#/c/14793/1/src/kudu/tserver/tablet_server-test.cc@2500
PS1, Line 2500: LOG(INFO) << "Scanning with a limit of " 
> As an alternative to continuing with next iteration, I think it's possible
OK



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia45372ff5e5460d220b7a3040de2e499541bf37f
Gerrit-Change-Number: 14793
Gerrit-PatchSet: 3
Gerrit-Owner: YangSong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: YangSong <[email protected]>
Gerrit-Comment-Date: Tue, 26 Nov 2019 02:24:21 +0000
Gerrit-HasComments: Yes

Reply via email to