fix: cpu-ram-crash#61
Conversation
Eliminates OOM crash when restoring databases larger than ~2 GB. std::io::copy uses a 64 KB buffer regardless of file size. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Same OOM fix applied to MySQL restore. std::io::copy streams with a 64 KB buffer, keeping heap flat for arbitrarily large SQL dumps. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe pull request adds resource constraints to the containerized Rust application and optimizes database restore operations. MariaDB and MySQL restore procedures now stream SQL files directly into their respective processes instead of buffering entire files in memory, reducing memory overhead during restoration. ChangesDatabase Restore Streaming Optimization
Container Resource Limits
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Summary by CodeRabbit