emkornfield commented on PR #48345: URL: https://github.com/apache/arrow/pull/48345#issuecomment-3634372373
Talked offline and wanted to capture notes on high-level changes: 1. For headers, lets try to reduce duplication with values already in the parquet header. 2. For remaining items in headers, lets try to be parsimonious with values (i.e. 4 bytes is probably overkill for enums) 3. Naming convention on files is off (use snake_case). 4. Given description of ALP, we probably want a top level encoding enum value for the 2 different modes of ALP. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
