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

[perf] - Optimize Chunk Size Handling for Improved I/O and Memory Efficiency #3134

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ahrav
Copy link
Collaborator

@ahrav ahrav commented Jul 31, 2024

Description:

This PR refines the chunk size handling mechanism in our file processing pipeline to optimize I/O operations and memory usage. The changes include updating the logic for determining chunk sizes based on file sizes, using appropriate thresholds, and setting chunk sizes that align better with common file sizes and I/O efficiency.

Bechmarks:

Screenshot 2024-07-07 at 4 45 10 PM

~6GB Filesystem Scan
BEFORE: (~49s)
Screenshot 2024-07-20 at 2 53 36 PM

AFTER: (~37s)
Screenshot 2024-07-20 at 2 51 59 PM

Checklist:

  • Tests passing (make test-community)?
  • Lint passing (make lint this requires golangci-lint)?

@ahrav ahrav marked this pull request as ready for review July 31, 2024 21:04
@ahrav ahrav requested a review from a team as a code owner July 31, 2024 21:04
@ahrav ahrav requested a review from a team July 31, 2024 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant