goutamadwant commented on PR #5538: URL: https://github.com/apache/datafusion-comet/pull/5538#issuecomment-6801114374
Thanks. I reproduced the buffer retention and have a local change that explicitly releases the compressed input buffer, including failure paths. I also noticed #7016 now shares decoded broadcasts through CometBroadcastValue. Should direct reads only apply when shared decoding is disabled, or should this path be reworked around that ownership first? I have kept the changes local while that direction is clarified. -- 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]
