Skip to content

Commit e91f260

Browse files
authored
address feedback.
1 parent 0dba068 commit e91f260

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

src/main/thrift/parquet.thrift

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -712,16 +712,16 @@ struct DictionaryPageHeader {
712712
}
713713

714714
/**
715-
* New page format allowing reading levels without decompressing the data
715+
* Alternate page format allowing reading levels without decompressing the data
716716
* Repetition and definition levels are uncompressed
717717
* The remaining section containing the data is compressed if is_compressed is true
718718
*
719719
* Implementation note - this header is not necessarily a strict improvement over
720720
* `DataPageHeader` (in particular the original header might provide better compression
721721
* in some scenarios). Page indexes require pages start and end at row boundaries regardless of which
722722
* page header is used.
723-
*
724-
* As of December 2025, most known Parquet readers can read pages using this header.
723+
* https://parquet.apache.org/docs/file-format/implementationstatus/ tracks implementation readers and
724+
* writers that support this version.
725725
**/
726726
struct DataPageHeaderV2 {
727727
/** Number of values, including NULLs, in this data page. **/
@@ -1264,8 +1264,10 @@ union EncryptionAlgorithm {
12641264
struct FileMetaData {
12651265
/** Version of this file
12661266
*
1267-
* Writers should always populate "1" for version. Readers should
1268-
* accept "1" and "2" interchangeably. All other versions are currently
1267+
* As of December 2025, there is no agreed upon consensus of what constitutes
1268+
* version 2 of the file. For maximum compatibility with readers, writers should
1269+
* always populate "1" for version. For maximum with writers compatibility,
1270+
* readers should accept "1" and "2" interchangeably. All other versions are
12691271
* reserved for potential future use-cases.
12701272
*/
12711273
1: required i32 version

0 commit comments

Comments
 (0)