julienledem commented on PR #1369:
URL: https://github.com/apache/parquet-java/pull/1369#issuecomment-2148684660
I don't remember all the context, but if this is completely wrong, I'd
rather deprecate the field and document it should not be used rather than
setting the value to zero.
Setting to zero has a few issues:
- it doesn't properly communicate that the field should not be used and can
be confusing
- it might break implementations that have been using this to find the
first page. Since setting it to zero doesn't improve the situation as it is
merely a different wrong value I'd rather we don't change the behavior until
the field has been removed at the end of the deprecation cycle.
What do other implementations put in this field?
--
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]
