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

update logical types table for EXPORT INTO PARQUET #17293

Open
jayshrivastava opened this issue Jun 16, 2023 · 0 comments
Open

update logical types table for EXPORT INTO PARQUET #17293

jayshrivastava opened this issue Jun 16, 2023 · 0 comments

Comments

@jayshrivastava
Copy link

Related PR: cockroachdb/cockroach#104234

I missed some details in the release note of the above PR. This table needs to be updated to reflect what the new library does.

  • For INT/INT8, we use the INT logical type. We can also add a note "Note: it signed and has bit width 64"
  • For UUID, the logical type should say UUID
  • We now support the TUPLE crdb type which has a nil logical type. For the parquet type we can say something similar as to what it says for arrays: "Encoded as a group; each tuple value is encoded as per the preceding types".
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant