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

[colocation] Refactor encoding / decoding of table IDs #3551

Closed
ndeodhar opened this issue Feb 4, 2020 · 1 comment
Closed

[colocation] Refactor encoding / decoding of table IDs #3551

ndeodhar opened this issue Feb 4, 2020 · 1 comment
Assignees
@ndeodhar @jaki
Labels
area/docdb YugabyteDB core features area/ysql Yugabyte SQL (YSQL)

Comments

@ndeodhar
Copy link
Contributor

ndeodhar commented Feb 4, 2020 •
edited by daniel-yb
Loading

For colocated tables, we use a pg table IDs for user YSQL tables and cotable IDs for system tables as docdb prefix. Currently, the code for encoding / decoding these is in docdb itself, which might make the code harder to manage in future.

We should refactor and add a higher level abstraction for encoding / decoding these table IDs.
@ndeodhar ndeodhar added area/ysql Yugabyte SQL (YSQL) area/docdb YugabyteDB core features labels Feb 4, 2020
@m-iancu m-iancu added this to YQL-beta Nov 30, 2021
@frozenspider
Copy link
Contributor

Colocation no longer uses table OIDs since #7378

@frozenspider frozenspider moved this to Done in YQL-beta Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ndeodhar ndeodhar

@jaki jaki

Labels
area/docdb YugabyteDB core features area/ysql Yugabyte SQL (YSQL)
Projects
YQL-beta
Status: Done
Milestone
No milestone
Development

No branches or pull requests
3 participants
@frozenspider @ndeodhar @jaki