Skip to content

Conversation

@dnhatn
Copy link
Member

@dnhatn dnhatn commented Mar 1, 2019

Since #39006 we should be able to complete a peer-recovery without waiting for pending indexing operations. Thus, the assertion in testDoNotWaitForPendingSeqNo should be updated from false to true.

Closes #39510

@dnhatn dnhatn added >test Issues or PRs that are addressing/adding tests :Distributed Indexing/Recovery Anything around constructing a new shard, either from a local or a remote source. v7.0.0 v6.7.0 v8.0.0 v7.2.0 labels Mar 1, 2019
@dnhatn dnhatn requested review from jasontedor and ywelsch March 1, 2019 23:18
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Mar 2, 2019
@DaveCTurner
Copy link
Contributor

Heads up that I muted this test in 9f130a5 so this PR should unmute it.

@dnhatn
Copy link
Member Author

dnhatn commented Mar 2, 2019

Thanks @DaveCTurner.

dnhatn pushed a commit that referenced this pull request Mar 4, 2019
dnhatn pushed a commit that referenced this pull request Mar 4, 2019
dnhatn pushed a commit that referenced this pull request Mar 4, 2019
@dnhatn dnhatn merged commit 7fe62d0 into elastic:master Mar 4, 2019
@dnhatn dnhatn deleted the test-wait-seqno branch March 4, 2019 13:59
@dnhatn
Copy link
Member Author

dnhatn commented Mar 4, 2019

Thanks @ywelsch

dnhatn added a commit that referenced this pull request Mar 4, 2019
Since #39006 we should be able to complete a peer-recovery without
waiting for pending indexing operations. Thus, the assertion in
testDoNotWaitForPendingSeqNo should be updated from false to true.

Closes #39510
dnhatn added a commit that referenced this pull request Mar 4, 2019
Since #39006 we should be able to complete a peer-recovery without
waiting for pending indexing operations. Thus, the assertion in
testDoNotWaitForPendingSeqNo should be updated from false to true.

Closes #39510
dnhatn added a commit that referenced this pull request Mar 4, 2019
Since #39006 we should be able to complete a peer-recovery without
waiting for pending indexing operations. Thus, the assertion in
testDoNotWaitForPendingSeqNo should be updated from false to true.

Closes #39510
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Mar 6, 2019
* 6.7: (39 commits)
  Remove beta label from CCR (elastic#39722)
  Rename retention lease setting (elastic#39719)
  Add Docker build type (elastic#39378)
  Use any index specified by .watches for Watcher (elastic#39541) (elastic#39706)
  Add documentation on remote recovery (elastic#39483)
  fix typo in synonym graph filter docs
  Removed incorrect ML YAML tests (elastic#39400)
  Improved Terms Aggregation documentation (elastic#38892)
  Fix Fuzziness#asDistance(String) (elastic#39643)
  Revert "unmute EvilLoggerTests#testDeprecatedSettings (elastic#38743)"
  Mute TokenAuthIntegTests.testExpiredTokensDeletedAfterExpiration (elastic#39690)
  Fix security index auto-create and state recovery race (elastic#39582)
  [DOCS] Sorts security APIs
  Check for .watches that wasn't upgraded properly (elastic#39609)
  Assert recovery done in testDoNotWaitForPendingSeqNo (elastic#39595)
  [DOCS] Updates API in Watcher transform context (elastic#39540)
  Fixing the custom object serialization bug in diffable utils. (elastic#39544)
  mute test
  SQL: Don't allow inexact fields for MIN/MAX (elastic#39563)
  Update release notes for 6.7.0
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Indexing/Recovery Anything around constructing a new shard, either from a local or a remote source. >test Issues or PRs that are addressing/adding tests v6.7.0 v7.0.0-rc2 v7.2.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RecoveryDuringReplicationTests.testDoNotWaitForPendingSeqNo failed

5 participants