fsaintjacques commented on a change in pull request #7089: URL: https://github.com/apache/arrow/pull/7089#discussion_r419460990
########## File path: cpp/src/parquet/properties.h ########## @@ -34,10 +34,14 @@ namespace parquet { +/// Control for data types in parquet. struct ParquetVersion { enum type { PARQUET_1_0, PARQUET_2_0 }; }; +/// Controls layout of data pages. Review comment: I'll clarify, I'm asking this to be put in a doc comment for whoever read this and doesn't know which one to use. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org