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

mirror: Avoid flushing session to disk on each i/o operation. #1909

Merged
merged 1 commit into from
Dec 5, 2016

Conversation

harshavardhana
Copy link
Member

@harshavardhana harshavardhana commented Dec 5, 2016

The problem was for each read from statusCh we were writing
to session file again and also syncing to disk. This adds
severe performance problems.

Fixes #1908

@codecov-io
Copy link

codecov-io commented Dec 5, 2016

Current coverage is 8.42% (diff: 0.00%)

Merging #1909 into master will decrease coverage by <.01%

@@            master     #1909   diff @@
========================================
  Files           77        77          
  Lines         6410      6412     +2   
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
  Hits           540       540          
- Misses        5739      5741     +2   
  Partials       131       131          

Powered by Codecov. Last update ce9d706...fe64a05

The problem was for each read from statusCh we were writing
to session file again and also syncing to disk. This adds
severe performance problems.

Fixes minio#1908
@harshavardhana harshavardhana changed the title mirror: Fix performance problem by using bufferred channels. mirror: Avoid flushing session to disk on each i/o operation. Dec 5, 2016
@harshavardhana harshavardhana merged commit 1e48d10 into minio:master Dec 5, 2016
@harshavardhana harshavardhana deleted the fix-performance branch December 5, 2016 21:53
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.

2 participants