RussellSpitzer commented on PR #13880: URL: https://github.com/apache/iceberg/pull/13880#issuecomment-3228981866
Previously it would just completely drop the reference to the previous vec so it didn't matter. (That's the leak). I'll raise a new PR with a fix. Also fun fact, we already have a test THAT WOULD FAIL if we were actually tracking memory usage. See TestParquetDictionaryEncodedVectorizedReads.testMixedDictionaryNonDictionaryReads. One sec and i'll have a new PR up to show this -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
