wgtmac commented on PR #88:
URL: https://github.com/apache/parquet-testing/pull/88#issuecomment-3047261563
Before merging, I think I still have some questions.
1. The parquet-cli cannot print `is_{max/min}_value_exact` yet so we still
don't know whether these fields are set correctly. We might need to extend C++
`ParquetFilePrinter` to print these values.
2. For writer implementations that ignore `is_{max/min}_value_exact` fields,
readers should always regard them as `false`. Should we add columns that
actually write `is_{max/min}_value_exact=true`? The point is that we may not
have any file with `is_{max/min}_value_exact=true` in the parquet-testing repo.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
