Hi Ed, Sorry for the late reply. I agree that we need to replace BINARY with BYTE_ARRAY to avoid confusion because FIXED_LENGTH_BYTE_ARRAY may also be regarded as BINARY.
Best, Gang On Fri, May 24, 2024 at 2:01 AM Ed Seidl <[email protected]> wrote: > Hi all, > > A question came up in the discussion of PARQUET-2474 [1] about the use > of 'binary' in the LogicalTypes.md file [2]. Is the use of 'binary' > where 'BYTE_ARRAY' is the physical type a holdover from an earlier > version of the spec? How would you all feel about cleaning this up and > using 'BYTE_ARRAY' exclusively? > > Thanks, > Ed > > [1] https://github.com/apache/parquet-format/pull/241 > [2] > https://github.com/apache/parquet-format/pull/241#discussion_r1602673993 >
