emkornfield commented on code in PR #535: URL: https://github.com/apache/parquet-format/pull/535#discussion_r2589956219
########## Encodings.md: ########## @@ -22,6 +22,11 @@ Parquet encoding definitions This file contains the specification of all supported encodings. +Some Parquet implementations distinguish encodings as "v1" and "v2". From +a specification perspective this distinction is considered meaningless. Writers may use any +encoding with both data page v1 and data page v2. Readers should lazily evaluate if they can +read a file (e.g. only error when required to a read a page with an unknown encoding). + Review Comment: I rephrased to remove version entirely as it might cause confusion. -- 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]
