Skip to content

Conversation

@Tim-Brooks
Copy link
Contributor

This commit adds high-level documenation about following an index using
ccr. This page is supposed to bridge the gap between the basic
getting-started guide and the lower-level api documenation pages.

@Tim-Brooks Tim-Brooks added >docs General docs changes :Distributed Indexing/CCR Issues around the Cross Cluster State Replication features v6.7.0 v8.0.0 v7.2.0 v7.0.0-beta1 labels Mar 6, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@Tim-Brooks
Copy link
Contributor Author

Based on a conversation with @ywelsch - I removed references to the index.soft_deletes.retention.operations and replaced with with index.soft_deletes.retention_lease.period.

Let me know if this does not make sense or if I got some descriptions wrong.

Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

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

Thanks for writing this up. I left some feedback.

=== Index following operation

Once the follower index is created, <<remote-recovery, remote recovery>> is
complete, and the shards are active, the normal following process begins. During
Copy link
Member

Choose a reason for hiding this comment

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

This overlaps some with the ccr-overview docs, perhaps link there?

I also wonder if these two docs (the new one being added here, and the overview doc) should be merged? I am asking this question out loud, I do not have a strong viewpoint one way or the other.

from the leader are governed by settings that can be configured in the
{ref}/ccr-put-follow.html[create follower request].

The {ref}/ccr-get-follow-stats.html[get follow stats api] can be used to monitor
Copy link
Member

Choose a reason for hiding this comment

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

Similarly, the content in this paragraph is covered in the overview docs.

@Tim-Brooks
Copy link
Contributor Author

Thanks for the feedback @jasontedor. I have opened #39936 as an alternative where I combine this information with the overview page. We can proceed with whichever approach we think is better. I believe I have also address your other comments.

@jasontedor
Copy link
Member

@tbrooks8 My vote is that we go with the proposal in #39936 indeed.

@Tim-Brooks
Copy link
Contributor Author

Tim-Brooks commented Mar 12, 2019

Closing in favor of #39936

@Tim-Brooks Tim-Brooks closed this Mar 12, 2019
@Tim-Brooks Tim-Brooks deleted the index_following_documentation branch December 18, 2019 14:49
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 >docs General docs changes v6.7.0 v7.0.0-beta1 v7.2.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants