raulcd commented on issue #44643: URL: https://github.com/apache/arrow/issues/44643#issuecomment-2478605419
Thanks @pitrou that was also what I thought but I am confused after some debugging because even if I decide to not rewind (just for testing purposes): ```diff - length_ -= 1; - offset -= 1; + // length_ -= 1; + // offset -= 1; + std::cout << "Let's not rewind" << '\n'; ``` The two builders still have a mismatch, below some logging from my tests when not rewinding: ``` Chunker::Extend length_before 0 length_after: 2601690 Chunker::Extend num_converted 2601690 Let's not rewind FinishChunk length_: 2601690 FinishChunk reserved_: 0 ToArray(int64_t length) length: 2601690 ToArray length: 2601690 item_builder_->length() 19530430 key_builder_->length(): 19530431 ``` -- 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]
