iemejia opened a new pull request, #577: URL: https://github.com/apache/parquet-format/pull/577
## Summary Fix pseudocode bugs, missing fields, and grammar issues in the Bloom filter and Page index documentation. ## Changes ### BloomFilter.md - Fix `block_check` pseudocode: `setBit` -> `isSet` (checking bits, not setting them) - Fix struct name to match thrift (BloomFilterHeader) - Include missing `bloom_filter_length` field in ColumnMetaData snippet - Normalize "Bloom filter" casing (proper noun) throughout - "64 bits version" -> "the 64-bit version" - Normalize "bit set" -> "bitset" (consistent with thrift) - Add terminal periods in thrift-style doc comments ### PageIndex.md - Fix "Blart Versenwald III" double-quote typo - Fix heading capitalization: "page index" -> "Page Index" - Fix article: "one data page per the retrieved column" -> "per retrieved column" - Add missing terminal period after parquet.thrift link ## Validation No semantic/behavioral changes to the format specification. All fixes are documentation-only. _Split from #572 for easier review._ -- 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]
