Skip to content

Commit f637b76

Browse files
committed
Add debug for Windows failures
1 parent faa36de commit f637b76

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

cpp/src/arrow/util/rle_encoding_test.cc

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -998,6 +998,9 @@ void CheckRoundTrip(const Array& data, int bit_width, bool spaced, int32_t parts
998998
ASSERT_GE(parts, 1);
999999
ASSERT_LE(parts, data_size);
10001000

1001+
ARROW_SCOPED_TRACE("bit_width = ", bit_width, ", spaced = ", spaced, ", data_size = ", data_size,
1002+
", buffer_size = ", buffer_size);
1003+
10011004
const value_type* data_values = static_cast<const ArrayType&>(data).raw_values();
10021005

10031006
// Encode the data into `buffer` using the encoder.
@@ -1008,13 +1011,15 @@ void CheckRoundTrip(const Array& data, int bit_width, bool spaced, int32_t parts
10081011
// Depending on `spaced` we treat nulls as regular values.
10091012
if (data.IsValid(i) || !spaced) {
10101013
bool success = encoder.Put(static_cast<uint64_t>(data_values[i]));
1011-
ASSERT_TRUE(success) << "Encoding failed in pos " << i;
1014+
ASSERT_TRUE(success) << "Encoding failed in pos " << i << ", current encoder len: " << encoder.len();
10121015
++encoded_values_size;
10131016
}
10141017
}
10151018
int encoded_byte_size = encoder.Flush();
10161019
ASSERT_EQ(encoded_values_size, data_values_count)
10171020
<< "All values input were not encoded successfully by the encoder";
1021+
ARROW_LOG(INFO) << "bit_width = " <<bit_width << ", data_size = " << data_size
1022+
<< ", buffer size = " << buffer_size << ", encoded_byte_size = " << encoded_byte_size;
10181023

10191024
// Now we verify batch read
10201025
RleBitPackedDecoder<value_type> decoder(buffer.data(), encoded_byte_size, bit_width);

0 commit comments

Comments
 (0)