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

Segment Large CSV Batch Processes #1475

Merged
merged 37 commits into from
Feb 4, 2025
Merged

Segment Large CSV Batch Processes #1475

merged 37 commits into from
Feb 4, 2025

Conversation

jpengst
Copy link
Contributor

@jpengst jpengst commented Jan 15, 2025

Summary

Batch processes, that can potentially have thousands of lines in the CSV, are segmented into batches of 50 to reduce number of objects that have jobs re-run multiple times on the same object.

Related Ticket

#2873

@jpengst jpengst changed the title Segment BP Draft: Segment BP Jan 15, 2025
@K8Sewell K8Sewell marked this pull request as draft January 22, 2025 19:47
@K8Sewell K8Sewell changed the title Draft: Segment BP Segment Large CSV Batch Processes Jan 22, 2025
@K8Sewell K8Sewell marked this pull request as ready for review January 28, 2025 20:39
@K8Sewell K8Sewell requested a review from martinlovell February 3, 2025 21:38
Copy link
Collaborator

@martinlovell martinlovell left a comment

Choose a reason for hiding this comment

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

Looks good

@jpengst jpengst merged commit 2a311eb into main Feb 4, 2025
4 checks passed
@jpengst jpengst deleted the 2873_SegmentBP branch February 4, 2025 17:33
K8Sewell added a commit that referenced this pull request Feb 20, 2025
jpengst pushed a commit that referenced this pull request Feb 20, 2025
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.

3 participants