pvary commented on pull request #2052:
URL: https://github.com/apache/hive/pull/2052#issuecomment-793690150


   > I don't think we should allow to change this config at session level. Idea 
was to have a single threadpool for txn heartbeating job. Btw initialization is 
also under the static method.
   
   Yeah, `DbTxnManager.initHeartbeatExecutorService()` is static-like - would 
it hurt to create the ExecutorService in a static block - using the global 
config, and use this service everywhere?


----------------------------------------------------------------
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]

Reply via email to