Skip to content

Ensure shrunk indices carry over version information from its source#22469

Merged
s1monw merged 1 commit intoelastic:masterfrom
s1monw:issues/22373
Jan 6, 2017
Merged

Ensure shrunk indices carry over version information from its source#22469
s1monw merged 1 commit intoelastic:masterfrom
s1monw:issues/22373

Conversation

@s1monw
Copy link
Copy Markdown
Contributor

@s1monw s1monw commented Jan 6, 2017

Today when an index is shrunk the version information is not carried over
from the source to the target index. This can cause major issues like mapping
incompatibilities for instance if an index from a previous major version is shrunk.

This commit ensures that all version information from the soruce index is preserved
when a shrunk index is created.

Closes #22373

Today when an index is shrunk the version information is not carried over
from the source to the target index. This can cause major issues like mapping
incompatibilities for instance if an index from a previous major version is shrunk.

This commit ensures that all version information from the soruce index is preserved
when a shrunk index is created.

Closes elastic#22373
@gmoskovicz
Copy link
Copy Markdown
Contributor

Sweet, thanks @s1monw

Copy link
Copy Markdown
Contributor

@ywelsch ywelsch left a comment

Choose a reason for hiding this comment

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

LGTM

@s1monw s1monw merged commit 79093e1 into elastic:master Jan 6, 2017
@s1monw s1monw deleted the issues/22373 branch January 6, 2017 15:37
s1monw added a commit that referenced this pull request Jan 6, 2017
…22469)

Today when an index is shrunk the version information is not carried over
from the source to the target index. This can cause major issues like mapping
incompatibilities for instance if an index from a previous major version is shrunk.

This commit ensures that all version information from the soruce index is preserved
when a shrunk index is created.

Closes #22373
s1monw added a commit that referenced this pull request Jan 6, 2017
…22469)

Today when an index is shrunk the version information is not carried over
from the source to the target index. This can cause major issues like mapping
incompatibilities for instance if an index from a previous major version is shrunk.

This commit ensures that all version information from the soruce index is preserved
when a shrunk index is created.

Closes #22373
@clintongormley clintongormley added :Data Management/Indices APIs DO NOT USE. Use ":Distributed/Indices APIs" or ":StorageEngine/Templates" instead. and removed :Internal labels Jan 10, 2017
s1monw added a commit that referenced this pull request Feb 21, 2017
…22469)

Today when an index is shrunk the version information is not carried over
from the source to the target index. This can cause major issues like mapping
incompatibilities for instance if an index from a previous major version is shrunk.

This commit ensures that all version information from the soruce index is preserved
when a shrunk index is created.

Closes #22373
@s1monw s1monw added v5.2.2 and removed v5.2.0 labels Feb 21, 2017
@s1monw
Copy link
Copy Markdown
Contributor Author

s1monw commented Feb 21, 2017

this PR was not pushed to 5.2.0 and 5.2.1 it will be in 5.2.2 if we release this branch again. Sorry for the inconvenience. I simply missed the backport for some reason. Thanks @gmoskovicz for pointing this out to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug critical :Data Management/Indices APIs DO NOT USE. Use ":Distributed/Indices APIs" or ":StorageEngine/Templates" instead. v5.1.2 v5.2.2 v6.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants