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

Write Snapshots directly to the blobstore stream #7637

Merged
merged 1 commit into from Sep 8, 2014

Conversation

s1monw
Copy link
Contributor

@s1monw s1monw commented Sep 8, 2014

Today we serialize the snashot metadata to a byte array and then copy
the byte array to a stream. Instead this commit moves the serialization
directly to the target stream without the intermediate representation.

@clintongormley clintongormley changed the title [STORE] Write Snapshots directly to the blobstore stream Snapshot/Restore: Write Snapshots directly to the blobstore stream Sep 8, 2014
@imotov
Copy link
Contributor

imotov commented Sep 8, 2014

LGTM

Today we serialize the snashot metadata to a byte array and then copy
the byte array to a stream. Instead this commit moves the serialization
directly to the target stream without the intermediate representation.

Closes elastic#7637
@s1monw s1monw merged commit 2619911 into elastic:master Sep 8, 2014
s1monw added a commit that referenced this pull request Sep 8, 2014
Today we serialize the snashot metadata to a byte array and then copy
the byte array to a stream. Instead this commit moves the serialization
directly to the target stream without the intermediate representation.

Closes #7637
@s1monw s1monw deleted the simplify_blob_store branch September 8, 2014 20:31
@s1monw s1monw removed the review label Sep 8, 2014
@clintongormley clintongormley added the :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs label Jun 7, 2015
@clintongormley clintongormley changed the title Snapshot/Restore: Write Snapshots directly to the blobstore stream Write Snapshots directly to the blobstore stream Jun 7, 2015
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.4.0.Beta1 v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants