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

Add sizeInBytes field #940

Merged
merged 4 commits into from
Jun 20, 2024
Merged

Conversation

kx-chen
Copy link
Contributor

@kx-chen kx-chen commented May 30, 2024

Summary

Add sizeInBytes field to SnapshotMetadata and ChunkInfo. Used to keep track of size of chunks in preparation for dynamic chunk sizes.

Requirements

Copy link

Thanks for the contribution! Before we can merge this, we need @kx-chen to sign the Salesforce Inc. Contributor License Agreement.

@kx-chen kx-chen force-pushed the kx-chen/add-size-metadata branch 3 times, most recently from a4aea70 to 6c7faae Compare June 4, 2024 18:48
@kx-chen kx-chen requested a review from bryanlb June 4, 2024 18:48
@kx-chen kx-chen force-pushed the kx-chen/add-size-metadata branch from 6c7faae to 0156958 Compare June 4, 2024 18:49
Used to keep track of size of chunks in prepartion for dynamic chunk sizes
@kx-chen kx-chen merged commit 3003e94 into slackhq:master Jun 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants