benibus commented on PR #184: URL: https://github.com/apache/parquet-format/pull/184#issuecomment-1767050894
> I've suggested the name `FLOAT_16` in the vote like we already have logical types `INT_8` etc I think this was only the convention for legacy `ConvertedType` enums. We could theoretically deviate from that since there are no sized integral logical types (they're all rolled into `INTEGER`/`IntType`). As for `BYTE_STREAM_SPLIT`, my feeling is that we'll _probably_ want it (for parity with `FLOAT`/`DOUBLE`, at least), but it could be added as a follow-up - along with an implementation + benchmarks, if necessary. There's also some ambiguity about whether to support BSS for `FixedLenByteArray` generally, which may warrant a separate discussion. -- 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: dev-unsubscr...@parquet.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
