wgtmac commented on code in PR #35825:
URL: https://github.com/apache/arrow/pull/35825#discussion_r1243028152
##########
cpp/src/parquet/arrow/arrow_reader_writer_test.cc:
##########
@@ -4587,6 +4647,24 @@ TEST_F(TestArrowReadDeltaEncoding, DeltaBinaryPacked) {
::arrow::AssertTablesEqual(*actual_table, *expect_table);
}
+TEST_F(TestArrowReadDeltaEncoding, DeltaBinaryPackedWithLargeBinaryVariant) {
Review Comment:
I'm sorry that I was thinking this is for DELTA_LENGTH_BYTE_ARRAY encoding.
Please remove this test as it does not relate to BYTE_ARRAY type. Sorry about
that again! I will make a follow change to make sure it is covered here.
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
