Hello Kudu Jenkins, Andrew Wong,

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

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

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

Change subject: [test] use full specification for RedoType elements
......................................................................

[test] use full specification for RedoType elements

My follow-up patch hit a conflict of KuduWriteOperation and RedoType
enums in fuzz-itest.cc:

  src/kudu/integration-tests/fuzz-itest.cc:235:3:
  error: ‘INSERT’ conflicts with a previous declaration
     INSERT,
     ^~~~~~
  In file included from src/kudu/tablet/tablet_replica.h:42,
                   from src/kudu/integration-tests/fuzz-itest.cc:67:
  src/kudu/tablet/ops/write_op.h:66:3: note: previous declaration 
‘kudu::tablet::WritePrivilegeType INSERT’
     INSERT,

This changelist addresses the issue.  I separated this change from the
follow-up patch for the ease of reviewing and tracking these changes
in the code.

There are no functional changes in this patch.

Change-Id: Id64eb45fd33bed84cb26d44151deda405547d24f
---
M src/kudu/integration-tests/fuzz-itest.cc
1 file changed, 16 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/91/17091/2
--
To view, visit http://gerrit.cloudera.org:8080/17091
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id64eb45fd33bed84cb26d44151deda405547d24f
Gerrit-Change-Number: 17091
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to