HeartSaVioR commented on pull request #28769: URL: https://github.com/apache/spark/pull/28769#issuecomment-642206766
> @HeartSaVioR do you mean we will deal with 1) here? I agree 2) might be separate. I think this PR is OK but would also fix the "known" leak on line 42 (see above) as there's no sense in allowing it. No I meant I'll probably try to fix the interface like I suggested in another PR (I filed a new issue). If we'd like to fix "known" leak, better to fix here directly. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
