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

change storage_am related catalog table main_manifest field type from… #210

Conversation

wangliang03
Copy link
Contributor

… uint32 to uint64,

and change the name from relid to relnode

we did the same change on storage_am, the main_manifest is a system catalog, storage_am has dependencies on this change.
storage_am will use the relfilenode as the identifier instead of relid, and currenlty relfilenode is uint64 and relid is uint32.

… uint32 to uint64,

and change the name from relid to relnode
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@tuhaihe
Copy link
Member

tuhaihe commented Sep 20, 2023

Hi, @wangliang03, thanks for your work! Please rewrite your commit message title and body following our commit convention. TYVM.

@my-ship-it my-ship-it merged commit e026fec into cloudberrydb:union_store_catalog Sep 21, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants