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

Change metadata file format #11507

Merged

Conversation

imotov
Copy link
Contributor

@imotov imotov commented Jun 4, 2015

No description provided.

@imotov imotov added v2.0.0-beta1 review :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs labels Jun 4, 2015
@clintongormley clintongormley changed the title Snapshot/Restore: change metadata file format Change metadata file format Jun 8, 2015
@jpountz
Copy link
Contributor

jpountz commented Jun 8, 2015

LGTM

@imotov imotov force-pushed the change-metadata-files-format-in-snapshots branch from 74ca8a2 to 831cfa5 Compare June 10, 2015 15:44
@imotov imotov merged commit 831cfa5 into elastic:master Jun 10, 2015
@imotov imotov removed the review label Jun 10, 2015
@s1monw
Copy link
Contributor

s1monw commented Jun 10, 2015

would it make sense to checksum the metadata while you are changing the format?

@imotov
Copy link
Contributor Author

imotov commented Jun 10, 2015

@s1monw yes, it makes perfect sense. I will add that.

@imotov
Copy link
Contributor Author

imotov commented Jun 10, 2015

@s1monw created #11589

@imotov imotov deleted the change-metadata-files-format-in-snapshots branch May 1, 2020 22:07
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 v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants