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

fix meta follower read remaining jobs #3347

Merged
merged 1 commit into from
Nov 24, 2021

Conversation

liwenhui-soul
Copy link
Contributor

What type of PR is this?

  • bug
  • feature
  • enhancement

What does this PR do?

Which issue(s)/PR(s) this PR relates to?

Special notes for your reviewer, ex. impact of this fix, etc:

Additional context:

Checklist:

  • Documentation affected (Please add the label if documentation needs to be modified.)
  • Incompatible (If it is incompatible, please describe it and add corresponding label.)
  • Need to cherry-pick (If need to cherry-pick to some branches, please label the destination version(s).)
  • Performance impacted: Consumes more CPU/Memory

Release notes:

Please confirm whether to reflect in release notes and how to describe:

                                                            `

@liwenhui-soul liwenhui-soul added the ready-for-testing PR: ready for the CI test label Nov 24, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #3347 (8ef8f3d) into master (6541438) will decrease coverage by 0.01%.
The diff coverage is 58.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3347      +/-   ##
==========================================
- Coverage   85.31%   85.30%   -0.02%     
==========================================
  Files        1289     1289              
  Lines      120073   120076       +3     
==========================================
- Hits       102442   102432      -10     
- Misses      17631    17644      +13     
Impacted Files Coverage Δ
src/meta/processors/job/JobManager.cpp 67.47% <33.33%> (-0.23%) ⬇️
src/graph/visitor/RewriteVisitor.cpp 76.37% <50.00%> (ø)
src/graph/validator/GroupByValidator.cpp 95.28% <100.00%> (ø)
src/graph/validator/MatchValidator.cpp 85.32% <100.00%> (ø)
src/common/datatypes/HostAddrOps-inl.h 72.97% <0.00%> (-5.41%) ⬇️
src/kvstore/Part.cpp 54.44% <0.00%> (-2.87%) ⬇️
src/storage/mutate/AddVerticesProcessor.cpp 85.64% <0.00%> (-2.40%) ⬇️
src/common/base/Status.h 88.63% <0.00%> (-2.28%) ⬇️
src/codec/RowReaderWrapper.cpp 78.57% <0.00%> (-1.85%) ⬇️
src/kvstore/DiskManager.cpp 79.01% <0.00%> (-1.24%) ⬇️
... and 20 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b7c6901...8ef8f3d. Read the comment docs.

@liwenhui-soul
Copy link
Contributor Author

#3345

@critical27 critical27 merged commit 683f773 into vesoft-inc:master Nov 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants