You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "YangSong (Code Review)" <ge...@cloudera.org> on 2019/11/26 02:16:08 UTC

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

Hello Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Grant Henke, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/14793

to look at the new patch set (#2).

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
---
M src/kudu/tserver/tablet_server-test.cc
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/93/14793/2
-- 
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: newpatchset
Gerrit-Change-Id: Ia45372ff5e5460d220b7a3040de2e499541bf37f
Gerrit-Change-Number: 14793
Gerrit-PatchSet: 2
Gerrit-Owner: YangSong <sy...@yeah.net>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)