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

Backport to 1.7 - Snapshot info should contain version of elasticsearch that created the snapshot #12162

Merged

Conversation

imotov
Copy link
Contributor

@imotov imotov commented Jul 9, 2015

Backport of #11985 to 1.7

This information was stored with the snapshot but wasn't available on the interface. Knowing the version of elasticsearch that created the snapshot can be useful to determine the minimal version of the cluster that is required in order to restore this snapshot.

Closes #11980

@imotov imotov added >enhancement review :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v1.7.0 labels Jul 9, 2015
@@ -260,6 +278,9 @@ public void readFrom(StreamInput in) throws IOException {
} else {
shardFailures = ImmutableList.of();
}
if(in.getVersion().onOrAfter(Version.V_1_7_0)) {
Copy link
Member

Choose a reason for hiding this comment

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

Super-minor, but missing a space between if and ( here.

@dakrone
Copy link
Member

dakrone commented Jul 9, 2015

LGTM, left two bikeshed comments.

…e snapshot

This information was stored with the snapshot but wasn't available on the interface. Knowing the version of elasticsearch that created the snapshot can be useful to determine the minimal version of the cluster that is required in order to restore this snapshot.

Closes elastic#11980
@imotov imotov force-pushed the issue-11980-add-version-to-snapshot-info-1.x branch from f9ed0b5 to cdfdd76 Compare July 9, 2015 18:44
@imotov imotov merged commit cdfdd76 into elastic:1.7 Jul 9, 2015
@imotov imotov removed the review label Jul 9, 2015
@imotov imotov deleted the issue-11980-add-version-to-snapshot-info-1.x branch May 1, 2020 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >enhancement v1.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants