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

To support delete in toss. #3374

Merged
merged 12 commits into from
Dec 29, 2021
Merged

To support delete in toss. #3374

merged 12 commits into from
Dec 29, 2021

Conversation

liuyu85cn
Copy link
Contributor

What type of PR is this?

  • bug
  • [X ] feature
  • enhancement

What does this PR do?

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

support delete out/in edges in TOSS.

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:

                                                            `

@liuyu85cn liuyu85cn force-pushed the toss-delete branch 2 times, most recently from 6da5429 to 89db998 Compare December 2, 2021 09:45
@liuyu85cn liuyu85cn marked this pull request as ready for review December 6, 2021 06:47
@liuyu85cn liuyu85cn added the ready-for-testing PR: ready for the CI test label Dec 6, 2021
@liuyu85cn liuyu85cn force-pushed the toss-delete branch 2 times, most recently from 421473b to 764eb93 Compare December 13, 2021 09:11
@codecov-commenter
Copy link

codecov-commenter commented Dec 20, 2021

Codecov Report

Merging #3374 (c75a7b4) into master (9a0fbb6) will increase coverage by 0.36%.
The diff coverage is 68.81%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3374      +/-   ##
==========================================
+ Coverage   85.40%   85.76%   +0.36%     
==========================================
  Files        1289     1319      +30     
  Lines      119798   128132    +8334     
==========================================
+ Hits       102317   109898    +7581     
- Misses      17481    18234     +753     
Impacted Files Coverage Δ
src/clients/storage/InternalStorageClient.cpp 0.00% <0.00%> (ø)
src/clients/storage/InternalStorageClient.h 100.00% <ø> (ø)
src/daemons/StorageDaemon.cpp 62.22% <ø> (-0.57%) ⬇️
src/kvstore/raftex/RaftPart.cpp 73.18% <ø> (-0.17%) ⬇️
src/mock/MockData.h 100.00% <ø> (ø)
src/storage/GraphStorageServiceHandler.cpp 79.26% <0.00%> (-3.02%) ⬇️
src/storage/InternalStorageServiceHandler.cpp 10.00% <0.00%> (-4.29%) ⬇️
.../storage/transaction/ChainAddEdgesProcessorLocal.h 40.00% <ø> (ø)
...orage/transaction/ChainAddEdgesProcessorRemote.cpp 0.00% <0.00%> (ø)
...storage/transaction/ChainAddEdgesProcessorRemote.h 0.00% <ø> (ø)
... and 242 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 1c2e1aa...c75a7b4. Read the comment docs.

panda-sheep
panda-sheep previously approved these changes Dec 27, 2021
Copy link
Contributor

@panda-sheep panda-sheep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job!

dummy copy from chain add edges processor

reguraly update

some update

may compile

add some simple UT

adjusting UT

add explain DeleteEdgesRequest

add some UT

clear reverse table early

expose storage interface

update license & fix gcc compile error

fix compile error

update license

fix chain interface bug

fix mem-leak

rename some processor

continue rename Processor

add some log for debug

print readable edge

debug keys when delete

print conflict key and scanned key

print rank

delPrime delete the wrong key from memory lock

fix raft may rollback want we scanned
Copy link
Contributor

@panda-sheep panda-sheep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job!

@yixinglu yixinglu merged commit 8c2e50f into vesoft-inc:master Dec 29, 2021
@Sophie-Xie Sophie-Xie added the doc affected PR: improvements or additions to documentation label Jan 4, 2022
@cooper-lzy cooper-lzy self-requested a review January 5, 2022 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc affected PR: improvements or additions to documentation ready for review ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants