aihuaxu commented on PR #481: URL: https://github.com/apache/parquet-format/pull/481#issuecomment-2599507994
> @aihuaxu I am not sure why we need this type in the Variant binary encoding. Doesn't this just duplicate the `binary` type? We don't need two different ways to store binary data. Yeah. You are right. The storage is the same as the binary since we can't have the length in the type description. Really no need to add that. We can just use binary to store fixed(L). cc @RussellSpitzer and @emkornfield -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@parquet.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@parquet.apache.org For additional commands, e-mail: issues-h...@parquet.apache.org
