Skip to content

Batch SCC repo_labor inserts to reduce occurrence of massive log lines#402

Open
MoralCode wants to merge 2 commits into
mainfrom
fix/massive_log_lines
Open

Batch SCC repo_labor inserts to reduce occurrence of massive log lines#402
MoralCode wants to merge 2 commits into
mainfrom
fix/massive_log_lines

Conversation

@MoralCode

Copy link
Copy Markdown
Contributor

Description
This removes a debug print statement and performs database insertions from scc in chunks using the configured chunk size (if available) or 1000 rows as a fallback.

This PR fixes #401

Notes for Reviewers
Not yet extensively tested.

Signed commits

  • Yes, I signed my commits.

Generative AI disclosure

Please select one option:

  • This contribution was NOT assisted or created by Generative AI tools.
  • This contribution was assisted or created by Generative AI tools.

If AI tools were used, please provide details below:
- What tools were used? Sonnet 4.6 Medium
- How were these tools used? diagnosis and tracing the issue
- Did you review these outputs before submitting this PR? only used for diagnosis. code was written by me by copying from other places in our code where we do similar batching.

this keeps memory use bounded and also reduces the size of the insert statement that is used

Signed-off-by: Adrian Edwards <[email protected]>
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.

Single log line is 30+ MB in size

1 participant