Skip to content

persist: count compaction fast-path eligible reqs#18380

Merged
pH14 merged 1 commit intoMaterializeInc:mainfrom
pH14:persist-count-compaction-fast-path
Mar 24, 2023
Merged

persist: count compaction fast-path eligible reqs#18380
pH14 merged 1 commit intoMaterializeInc:mainfrom
pH14:persist-count-compaction-fast-path

Conversation

@pH14
Copy link
Contributor

@pH14 pH14 commented Mar 24, 2023

Seeing how much compaction work / S3 PUTs we could shave off if we reintroduce the compaction fast path for single non-empty batches (with 1 run and were written by compaction)

Motivation

Tips for reviewer

Checklist

  • This PR has adequate test coverage / QA involvement has been duly considered.
  • This PR has an associated up-to-date design doc, is a design doc (template), or is sufficiently small to not require a design.
  • This PR evolves an existing $T ⇔ Proto$T mapping (possibly in a backwards-incompatible way) and therefore is tagged with a T-proto label.
  • If this PR will require changes to cloud orchestration, there is a companion cloud PR to account for those changes that is tagged with the release-blocker label (example).
  • This PR includes the following user-facing behavior changes:

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants