Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/14793 )

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

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

'kMaxLimit' may be 0, then an exception reported like 'Floating point 
exception'.
Here we cannot mod 0, if 'KMaxLimit' is 0, we're not taking mod any
more, we set 'kLimit' to 0 directly.

Change-Id: Ia45372ff5e5460d220b7a3040de2e499541bf37f
Reviewed-on: http://gerrit.cloudera.org:8080/14793
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <[email protected]>
Reviewed-by: Grant Henke <[email protected]>
---
M src/kudu/tserver/tablet_server-test.cc
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved
  Grant Henke: Looks good to me, approved

--
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: merged
Gerrit-Change-Id: Ia45372ff5e5460d220b7a3040de2e499541bf37f
Gerrit-Change-Number: 14793
Gerrit-PatchSet: 4
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]>

Reply via email to