Skip to content

Conversation

@dnhatn
Copy link
Member

@dnhatn dnhatn commented Nov 8, 2019

Throw an appropriate error message when the follower index is not found or is a regular index.

See: https://discuss.elastic.co/t/no-shard-follow-tasks-when-pausing-following-on-follower-index/207074/3

@dnhatn dnhatn added >enhancement :Distributed Indexing/CCR Issues around the Cross Cluster State Replication features v8.0.0 v7.6.0 labels Nov 8, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (:Distributed/CCR)

Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

LGTM

@martijnvg
Copy link
Member

(The IndexFollowingIT#testUnfollowNonExistingIndex() does also need to be adjusted)

@dnhatn
Copy link
Member Author

dnhatn commented Nov 11, 2019

Thanks @martijnvg.

@dnhatn dnhatn merged commit ae0d0ca into elastic:master Nov 11, 2019
@dnhatn dnhatn deleted the improve-error-message-pause-ccr branch November 11, 2019 13:57
dnhatn added a commit that referenced this pull request Nov 20, 2019
Throw an appropriate error message when the follower index is not found
or is a regular index.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Indexing/CCR Issues around the Cross Cluster State Replication features >enhancement v7.6.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants