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.

Sign up for GitHub

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

Jump to bottom

[dingo-exec] Add table id to the key to access the row store. #41

Closed
lasyard opened this issue Feb 10, 2022 · 0 comments
Closed

[dingo-exec] Add table id to the key to access the row store. #41

lasyard opened this issue Feb 10, 2022 · 0 comments
Assignees
@lasyard
Labels
enhancement New feature or request

Comments

@lasyard
Copy link
Member

lasyard commented Feb 10, 2022

Currently only the primary fields of a table are used to construct the key of the kv store. If the store is a whole kv store, the keys of different table will be mixed up. So a fixed-length table id should be prepend to the key to identify data of the table.
@lasyard lasyard added the enhancement New feature or request label Feb 10, 2022
@lasyard lasyard self-assigned this Feb 10, 2022
astor-oss pushed a commit that referenced this issue Mar 4, 2022
@lasyard @astor-oss
[dingo-exec] Fixed #41: Add table id to the key to access the row sto…
8fecbe1 
…re. (#42)

contains these modifications:

1. Generify tableName to TableId to transfer non-string table identification.

2. Added table id prefix to access row store.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lasyard lasyard

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lasyard