Neuw84 commented on PR #17296: URL: https://github.com/apache/iceberg/pull/17296#issuecomment-5103444921
Yes, it had exactly the same leak, and it is fixed in this PR. The two readers are changed symmetrically. `LastUpdatedSeqVectorReader` also allocated a fresh `BigIntVector` from the root allocator on every `read` (`allocateBigIntVector(LAST_UPDATED_SEQ, numValsToRead)`), kept no reference to it and had a no-op `close()`, so it leaked one vector per batch whenever `_last_updated_sequence_number` was projected. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
