LuciferYang commented on pull request #35262: URL: https://github.com/apache/spark/pull/35262#issuecomment-1025369374
> Oh dear. This will take some fixing. Thank you for catching this! I think we can also remove the `OFF_HEAP` mode support in current pr, (Sorry, I haven't seen the advantages of `OFF_HEAP` mode in the current scenario and other readers haven't used `OFF_HEAP` mode yet) and add the unified resource release mechanism and `OFF_HEAP` mode support in another pr. This may help to make the current pr merge faster. -- 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]
