You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2017/06/23 22:31:38 UTC
[kudu-CR] [raft consensus-itest] disable TestHammerOneRow for a while
Alexey Serbin has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7285
Change subject: [raft_consensus-itest] disable TestHammerOneRow for a while
......................................................................
[raft_consensus-itest] disable TestHammerOneRow for a while
Changelist c25d127ad3512ed9c9a2565681ffce9de639639f exposed inconsistent
verification of SetFlag() results in the EnableLogLatency() function.
This patch fixed contains the fix for that inconsistency from Adar
and it also disables the TestHammerOneRow scenario since it started
failing as a result of the changelist mentioned above.
The follow-up patch will re-enable the TestHammerOneRow scenario
after investigating the underlying problem.
Change-Id: I9759f7e77eba3ed894e8f9a729f729595635c334
---
M src/kudu/integration-tests/raft_consensus-itest.cc
1 file changed, 3 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/85/7285/1
--
To view, visit http://gerrit.cloudera.org:8080/7285
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9759f7e77eba3ed894e8f9a729f729595635c334
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
[kudu-CR] [raft consensus-itest] disable TestHammerOneRow for a while
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has submitted this change and it was merged.
Change subject: [raft_consensus-itest] disable TestHammerOneRow for a while
......................................................................
[raft_consensus-itest] disable TestHammerOneRow for a while
Changelist c25d127ad3512ed9c9a2565681ffce9de639639f exposed inconsistent
verification of SetFlag() results in the EnableLogLatency() function.
This patch contains the fix from Adar for that inconsistency
and it also disables the TestHammerOneRow scenario since it started
failing as a result of the changelist mentioned above.
The follow-up patch will re-enable the TestHammerOneRow scenario
after investigating the underlying problem.
Change-Id: I9759f7e77eba3ed894e8f9a729f729595635c334
Reviewed-on: http://gerrit.cloudera.org:8080/7285
Reviewed-by: Adar Dembo <ad...@cloudera.com>
Reviewed-by: Todd Lipcon <to...@apache.org>
Tested-by: Kudu Jenkins
---
M src/kudu/integration-tests/raft_consensus-itest.cc
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
Adar Dembo: Looks good to me, approved
Todd Lipcon: Looks good to me, approved
Kudu Jenkins: Verified
--
To view, visit http://gerrit.cloudera.org:8080/7285
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9759f7e77eba3ed894e8f9a729f729595635c334
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <da...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
[kudu-CR] [raft consensus-itest] disable TestHammerOneRow for a while
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has uploaded a new patch set (#2).
Change subject: [raft_consensus-itest] disable TestHammerOneRow for a while
......................................................................
[raft_consensus-itest] disable TestHammerOneRow for a while
Changelist c25d127ad3512ed9c9a2565681ffce9de639639f exposed inconsistent
verification of SetFlag() results in the EnableLogLatency() function.
This patch contains the fix from Adar for that inconsistency
and it also disables the TestHammerOneRow scenario since it started
failing as a result of the changelist mentioned above.
The follow-up patch will re-enable the TestHammerOneRow scenario
after investigating the underlying problem.
Change-Id: I9759f7e77eba3ed894e8f9a729f729595635c334
---
M src/kudu/integration-tests/raft_consensus-itest.cc
1 file changed, 3 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/85/7285/2
--
To view, visit http://gerrit.cloudera.org:8080/7285
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9759f7e77eba3ed894e8f9a729f729595635c334
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <da...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
[kudu-CR] [raft consensus-itest] disable TestHammerOneRow for a while
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has posted comments on this change.
Change subject: [raft_consensus-itest] disable TestHammerOneRow for a while
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7285
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9759f7e77eba3ed894e8f9a729f729595635c334
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <da...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: No
[kudu-CR] [raft consensus-itest] disable TestHammerOneRow for a while
Posted by "Adar Dembo (Code Review)" <ge...@cloudera.org>.
Adar Dembo has posted comments on this change.
Change subject: [raft_consensus-itest] disable TestHammerOneRow for a while
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7285
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9759f7e77eba3ed894e8f9a729f729595635c334
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <da...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: No