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

Dss part1: use local id in the space range for tag edge index #2550

Merged
merged 2 commits into from
Aug 25, 2021

Conversation

panda-sheep
Copy link
Contributor

@panda-sheep panda-sheep commented Aug 23, 2021

As title

1) Tag, edge, tag index and edge index use space range id.
2) Compatible with the existing old schema creating previously.

Synchronize nebula-storage repo pr vesoft-inc/nebula-storage#501

@Shylock-Hg
Copy link
Contributor

Why need this?

@panda-sheep
Copy link
Contributor Author

panda-sheep commented Aug 23, 2021

Why need this?

Data synchronization between clusters

Copy link
Contributor

@critical27 critical27 left a comment

Choose a reason for hiding this comment

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

Reviewed before, LGTM.

@CLAassistant
Copy link

CLAassistant commented Aug 24, 2021

CLA assistant check
All committers have signed the CLA.

@Shylock-Hg Shylock-Hg added the incompatible PR: incompatible with the recently released version label Aug 24, 2021
@panda-sheep panda-sheep removed the incompatible PR: incompatible with the recently released version label Aug 24, 2021
@panda-sheep
Copy link
Contributor Author

Please see the second, compatible with old data

@Shylock-Hg
Copy link
Contributor

Shylock-Hg commented Aug 24, 2021

So now there are independent incremental id in separate spaces, the schema id will be same for different spaces, and this behavior different from before. For more compatible I think you could reuse the Global ID and just store them in the local id key to distinguish the owned space, and in this case the schema id in different space won't be same (they are still generated by global id)

@critical27 critical27 merged commit 1525aa3 into vesoft-inc:master Aug 25, 2021
@panda-sheep panda-sheep deleted the use_local_id branch August 25, 2021 02:39
@panda-sheep panda-sheep changed the title use local id in the space range for tag edge index Dss part1: use local id in the space range for tag edge index Nov 1, 2021
@panda-sheep panda-sheep mentioned this pull request Jan 18, 2022
11 tasks
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Sep 14, 2023
Fix tagnum is not passed into GITHUB_OUTPUT

Co-authored-by: George <58841610+Shinji-IkariG@users.noreply.github.com>
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.

7 participants