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

meta: increase inodeBatch & sliceIdBatch #4188

Merged
merged 1 commit into from
Nov 23, 2023
Merged

meta: increase inodeBatch & sliceIdBatch #4188

merged 1 commit into from
Nov 23, 2023

Conversation

SandyXSD
Copy link
Contributor

No description provided.

@SandyXSD SandyXSD requested a review from davies November 23, 2023 04:06
Copy link

codecov bot commented Nov 23, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (e242ca5) 55.61% compared to head (25743f2) 55.61%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4188   +/-   ##
=======================================
  Coverage   55.61%   55.61%           
=======================================
  Files         154      154           
  Lines       39492    39492           
=======================================
  Hits        21965    21965           
+ Misses      15074    15073    -1     
- Partials     2453     2454    +1     

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

@davies davies merged commit 9a40759 into main Nov 23, 2023
@davies davies deleted the increase-batch branch November 23, 2023 08:53
@eryugey
Copy link
Contributor

eryugey commented Nov 28, 2023

Any reason we're doing this change? What's the problem it wants to solve? I'd be better to add more info in commit log, so reviewers and/or users from community could know the background and why the change is needed.

@davies
Copy link
Contributor

davies commented Nov 29, 2023

This patch can reduce the number transaction needed and reduce the conflict.

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.

3 participants