wgtmac commented on code in PR #33776:
URL: https://github.com/apache/arrow/pull/33776#discussion_r1088416807
##########
cpp/src/parquet/parquet.thrift:
##########
@@ -585,15 +585,15 @@ union BloomFilterAlgorithm {
/** Hash strategy type annotation. xxHash is an extremely fast
non-cryptographic hash
* algorithm. It uses 64 bits version of xxHash.
**/
-struct XxHash {}
+struct XxHasher {}
Review Comment:
Should we update the generated cpp as well? And is it good time to sync the
latest thrift file from upstream parquet-format? @wjones127 @pitrou
--
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]
