Skip to content
This repository has been archived by the owner on Feb 8, 2024. It is now read-only.

feat: add partition limiter + overflow #48

Merged
merged 10 commits into from
Nov 13, 2023
Merged

feat: add partition limiter + overflow #48

merged 10 commits into from
Nov 13, 2023

Conversation

ellie
Copy link
Contributor

@ellie ellie commented Nov 13, 2023

No description provided.

@ellie ellie changed the title [draft] feat: add partition limiter + overflow feat: add partition limiter + overflow Nov 13, 2023
@ellie ellie requested a review from xvello November 13, 2023 14:17
@ellie ellie marked this pull request as ready for review November 13, 2023 14:17
@ellie ellie force-pushed the ellie/partition-limiter branch from f3f3f51 to e85ac06 Compare November 13, 2023 14:30
xvello
xvello approved these changes Nov 13, 2023
View reviewed changes
Copy link
Contributor

@xvello xvello left a comment

Choose a reason for hiding this comment

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

Great, two nits

capture/src/partition_limits.rs Outdated Show resolved Hide resolved
capture-server/tests/events.rs Outdated Show resolved Hide resolved
Ellie Huxtable and others added 3 commits November 13, 2023 14:39
@ellie ellie merged commit 014456d into main Nov 13, 2023
4 checks passed
@ellie ellie deleted the ellie/partition-limiter branch November 13, 2023 15:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@xvello xvello xvello approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ellie @xvello