Skip to content

Conversation

@original-brownbear
Copy link
Contributor

Adds package level JavaDoc for snapshot clones.
Relates #61839

Adds package level JavaDoc for snapshot clones.
Relates elastic#61839
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (:Distributed/Snapshot/Restore)

@elasticmachine elasticmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label Oct 3, 2020
.collect(Collectors.toMap(IndexId::getName, Function.identity()));
}

// TODO: It is worth revisiting the design choice of creating a placeholder entry in snapshots-in-progress here once we have a cache
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just a thought I had while writing this up. The placeholder logic might be questionable but we can easily remove it in a BwC manner like we did for snapshot create if want to in a follow-up so I'm not too worried here either.

Copy link
Member

@tlrx tlrx left a comment

Choose a reason for hiding this comment

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

LGTM, very well written doc 👍

@original-brownbear
Copy link
Contributor Author

Thanks Tanguy!

@original-brownbear original-brownbear merged commit f802663 into elastic:master Oct 5, 2020
@original-brownbear original-brownbear deleted the final-touches-clone-api branch October 5, 2020 10:39
original-brownbear added a commit to original-brownbear/elasticsearch that referenced this pull request Oct 5, 2020
Adds package level JavaDoc for snapshot clones.
Relates elastic#61839
original-brownbear added a commit that referenced this pull request Oct 5, 2020
Snapshot clone API. Complete except for some TODOs around documentation (and adding HLRC support).

backport of #61839, #63217, #63037
@original-brownbear original-brownbear restored the final-touches-clone-api branch December 6, 2020 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >non-issue Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. v7.10.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants