Skip to content

fix: reduce DB and log bloat by not logging writePack content#1389

Open
kriswest wants to merge 3 commits intomainfrom
1388-reduce-db-bloat
Open

fix: reduce DB and log bloat by not logging writePack content#1389
kriswest wants to merge 3 commits intomainfrom
1388-reduce-db-bloat

Conversation

@kriswest
Copy link
Contributor

@kriswest kriswest commented Feb 6, 2026

resolves #1388

Reduces bloat of DB records and server logs (by 13x!) by not logging a byte array encoding the git index files created in the writePack step.

Signed-off-by: Kris West <kristopher.west@natwest.com>
@netlify
Copy link

netlify bot commented Feb 6, 2026

Deploy Preview for endearing-brigadeiros-63f9d0 canceled.

Name Link
🔨 Latest commit bf516a9
🔍 Latest deploy log https://app.netlify.com/projects/endearing-brigadeiros-63f9d0/deploys/69864b09a278200008220b8f

@kriswest kriswest requested a review from a team February 6, 2026 19:29
@github-actions github-actions bot added the fix label Feb 6, 2026
@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Signed-off-by: Kris West <kristopher.west@natwest.com>
@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.34%. Comparing base (c206394) to head (bf516a9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1389      +/-   ##
==========================================
- Coverage   81.35%   81.34%   -0.01%     
==========================================
  Files          65       65              
  Lines        4649     4648       -1     
  Branches      792      792              
==========================================
- Hits         3782     3781       -1     
  Misses        852      852              
  Partials       15       15              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kriswest kriswest mentioned this pull request Feb 6, 2026
19 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

writePack unnecessarily bloats the database and logs

1 participant