Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Properly set parallelism #7370

Merged
merged 1 commit into from
Dec 9, 2019
Merged
Rate limit · GitHub

Access has been restricted

You have triggered a rate limit.

Please wait a few minutes before you try again;
in some cases this may take up to an hour.

Conversation

carllin
Copy link
Contributor

@carllin carllin commented Dec 9, 2019

Problem

Rocks no longer needs to explicitly set the number of threads for flush/compaction, should be handled by default

Summary of Changes

Remove explicitly setting flush/compaction threads.
Fixes #

@carllin carllin added the v0.21 label Dec 9, 2019
@carllin carllin requested a review from sagar-solana December 9, 2019 07:41
@codecov
Copy link

codecov bot commented Dec 9, 2019

Codecov Report

Merging #7370 into master will decrease coverage by <.1%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #7370     +/-   ##
========================================
- Coverage    80.4%   80.4%   -0.1%     
========================================
  Files         244     244             
  Lines       47617   47615      -2     
========================================
- Hits        38297   38295      -2     
  Misses       9320    9320

@carllin carllin merged commit e0600e5 into solana-labs:master Dec 9, 2019
mergify bot pushed a commit that referenced this pull request Dec 9, 2019
(cherry picked from commit e0600e5)
solana-grimes pushed a commit that referenced this pull request Dec 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant