Skip to content

Coalesce per-partition fanout and bounding concurrent scans into single arm#5095

Open
tillrohrmann wants to merge 1 commit into
restatedev:mainfrom
tillrohrmann:simplify-partitioned-scans
Open

Coalesce per-partition fanout and bounding concurrent scans into single arm#5095
tillrohrmann wants to merge 1 commit into
restatedev:mainfrom
tillrohrmann:simplify-partitioned-scans

Conversation

@tillrohrmann

Copy link
Copy Markdown
Contributor

When bounding the number of concurrent scans due to a scan operation which touches many partition keys, we can still use the logic for per-partition fanout to limit the partition key range on a given partition. This simplifies the logic a little bit and makes the processing a little bit more efficient as we don't have to scan the whole partition key range in some cases.

…le arm

When bounding the number of concurrent scans due to a scan operation which touches many
partition keys, we can still use the logic for per-partition fanout to limit the partition
key range on a given partition. This simplifies the logic a little bit and makes the
processing a little bit more efficient as we don't have to scan the whole partition key range
in some cases.

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@tillrohrmann

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🚀

Reviewed commit: 18ee51b935

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

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.

1 participant