Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix IndexShardTestCase.recoverReplica(IndexShard, IndexShard, boolean) #37414

Merged
merged 1 commit into from
Jan 15, 2019

Conversation

tlrx
Copy link
Member

@tlrx tlrx commented Jan 14, 2019

This pull requests fixes the IndexShardTestCase.recoverReplica(IndexShard, IndexShard, boolean) method where the startReplica was not correctly propagated and the value true always used instead.

@tlrx tlrx added >test Issues or PRs that are addressing/adding tests v7.0.0 :Distributed/Engine Anything around managing Lucene and the Translog in an open shard. v6.7.0 labels Jan 14, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@tlrx
Copy link
Member Author

tlrx commented Jan 14, 2019

@elasticmachine run gradle build tests 2

1 similar comment
@tlrx
Copy link
Member Author

tlrx commented Jan 15, 2019

@elasticmachine run gradle build tests 2

@tlrx tlrx merged commit 23ae980 into elastic:master Jan 15, 2019
@tlrx tlrx deleted the fix-recoverfromreplica branch January 15, 2019 11:48
@tlrx
Copy link
Member Author

tlrx commented Jan 15, 2019

Thanks @ywelsch

tlrx added a commit that referenced this pull request Jan 15, 2019
#37414)

This commit fixes the IndexShardTestCase.recoverReplica(IndexShard, IndexShard, boolean) 
method where the startReplica parameter was not correctly propagated and the value
 true always used instead.
tlrx added a commit that referenced this pull request Jan 15, 2019
#37414)

This commit fixes the IndexShardTestCase.recoverReplica(IndexShard, IndexShard, boolean) 
method where the startReplica parameter was not correctly propagated and the value
 true always used instead.
@tlrx tlrx added the v6.6.1 label Jan 15, 2019
tlrx added a commit that referenced this pull request Jan 15, 2019
#37414)

This commit fixes the IndexShardTestCase.recoverReplica(IndexShard, IndexShard, boolean) 
method where the startReplica parameter was not correctly propagated and the value
 true always used instead.
@tlrx tlrx added the v6.5.5 label Jan 15, 2019
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Jan 15, 2019
* elastic/master:
  Docs be explicit on how to turn off deprecated auditing (elastic#37316)
  Fix line length for monitor and remove suppressions (elastic#37456)
  Fix IndexShardTestCase.recoverReplica(IndexShard, IndexShard, boolean) (elastic#37414)
  Update the Flush API documentation (elastic#33551)
  [TEST] Muted testDifferentRolesMaintainPathOnRestart
  Remove dead code from ShardSearchStats (elastic#37421)
  Simplify testSendSnapshotSendsOps (elastic#37445)
  SQL: Fix issue with field names containing "." (elastic#37364)
  Restore lost @Inject annotation (elastic#37452)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed/Engine Anything around managing Lucene and the Translog in an open shard. >test Issues or PRs that are addressing/adding tests v6.5.5 v6.6.1 v6.7.0 v7.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants