[GitHub] [iceberg] RussellSpitzer edited a comment on pull request #4126: Spark: Close executor services after action completes in Procedures

[GitBox <gi...@apache.org>]

RussellSpitzer edited a comment on pull request #4126:
URL: https://github.com/apache/iceberg/pull/4126#issuecomment-1042043496


   Reading the comments on the JavaDocs for the ThreadPoolExecutor, I was wrong the behavior changes between Java 8 and -11. Is it wrong that I still think about Java 8 as the current java?
   
   
   ```
       /**
        * @implNote Previous versions of this class had a finalize method
        * that shut down this executor, but in this version, finalize
        * does nothing.
        */
       @Deprecated(since="9")
       protected void finalize() {}
   ```


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org