Skip to content

Conversation

@bleskes
Copy link
Contributor

@bleskes bleskes commented Nov 22, 2015

The work for #10708 requires tighter integration with the current shard routing of a shard. As such, we need to make sure it is set before the IndexService exposes the shard to external operations.

…operations

The work for elastic#10708 requires tighter integration with the current shard routing of a shard. As such, we need to make sure it is set before the IndexService exposes the shard to external operations.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this not relevant anymore?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's done in the createShard call.

@ywelsch
Copy link
Contributor

ywelsch commented Nov 23, 2015

LGTM

@bleskes bleskes closed this in 6e2e91c Nov 23, 2015
bleskes added a commit that referenced this pull request Nov 23, 2015
…operations

The work for #10708 requires tighter integration with the current shard routing of a shard. As such, we need to make sure it is set before the IndexService exposes the shard to external operations.

Closes #14918
@s1monw
Copy link
Contributor

s1monw commented Nov 23, 2015

good, thanks!

@clintongormley clintongormley added :Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. and removed :Cluster labels Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. >enhancement v2.2.0 v5.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants