ILuffZhe commented on pull request #2698:
URL: https://github.com/apache/calcite/pull/2698#issuecomment-1018585102


   - One small reminder: the commit info should be like "[CALCITE-4992] 
RestClient resource leak in ElasticSearch adapter", which can be linked to JIRA 
automatically
   
   -  Could you please remove the unnecessary blank lines?
   
   -  Instead of closing the RestClient by JVM GC, how about initiatively 
closing the client after each query? 


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


Reply via email to