Hello Alexey Serbin,

I'd like you to do a code review. Please visit

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

to review the following change.


Change subject: [compile] fix a compile warning
......................................................................

[compile] fix a compile warning

I noticed that there is a warning during the compile
process:
`
/data/github-kudu/kudu/src/kudu/integration-tests/auto_incrementing-itest.cc: 
In member function ‘virtual void 
kudu::itest::AutoIncrementingItest_BootstrapWalsDiverge_Test::TestBody()’:
/data/github-kudu/kudu/src/kudu/integration-tests/auto_incrementing-itest.cc:530:7:
 warning: operation on ‘j’ may be undefined [-Wsequence-point]
     j = ++j % kNumTabletServers;
     ~~^~~~~~~~~~~~~~~~~~~~~~~~~
`
Although it does not involve functional defects, it
still needs to be fixed.

As it does not involve any logic changes, no new unit
tests have been added.

Change-Id: I125674ad3bfcc14ae5077b84d6f3d01fcacd1e08
Reviewed-on: http://gerrit.cloudera.org:8080/20182
Reviewed-by: Alexey Serbin <ale...@apache.org>
Tested-by: Alexey Serbin <ale...@apache.org>
(cherry picked from commit 35aa458d7a5f67caf0ef4111408885fa5ba0b532)
---
M src/kudu/integration-tests/auto_incrementing-itest.cc
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/22/20222/1
-- 
To view, visit http://gerrit.cloudera.org:8080/20222
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.17.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I125674ad3bfcc14ae5077b84d6f3d01fcacd1e08
Gerrit-Change-Number: 20222
Gerrit-PatchSet: 1
Gerrit-Owner: Yingchun Lai <laiyingc...@apache.org>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: KeDeng <kdeng...@gmail.com>

Reply via email to