mapleFU commented on code in PR #41117:
URL: https://github.com/apache/arrow/pull/41117#discussion_r1559814080
##########
cpp/src/arrow/io/compressed.cc:
##########
@@ -353,7 +356,9 @@ class CompressedInputStream::Impl {
// Try to feed more data into the decompressed_ buffer.
Status RefillDecompressed(bool* has_data) {
Review Comment:
Before https://github.com/apache/arrow/pull/39807 . `decompressed_` would
set to nullptr if it's all consumed. And even
https://github.com/apache/arrow/pull/39807 change the syntax, if would gurantee
ok since `DecompressData()` in `compressed_` and `decompressor_` would protect
it. So I think this just didn't change the syntax
--
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]
