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

Partition batches with greedy algorithm #224

Merged
merged 1 commit into from
Mar 23, 2022
Merged

Conversation

parasj
Copy link
Contributor

@parasj parasj commented Mar 22, 2022

Fixes #131

Also parallelizes copying logs from gateways to the client.

@parasj parasj self-assigned this Mar 22, 2022
@parasj parasj added client bug Something isn't working labels Mar 22, 2022
@sarahwooders sarahwooders merged commit c6dbe88 into main Mar 23, 2022
@sarahwooders sarahwooders deleted the dev/paras/fix_131 branch March 23, 2022 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug] replicate-json broken by some plans: AssertionError: 2 batches, expected 3
2 participants