[CALCITE-4992] RestClient resource leak in Elasticsearch adapter

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.29.0
• Fix Version/s: 1.31.0
• Component/s: elasticsearch-adapter
• Labels: pull-request-available
• Assignee: James Turton
• Reporter: James Turton
• Created: 21/Jan/22 14:28
• Updated: 03/Aug/22 16:21
• Resolved: 13/Apr/22 17:29

Description

RestClient instances should receive a `close()`call at the end of their lifecyle, as documented here.

https://www.elastic.co/guide/en/elasticsearch/client/java-rest/current/java-rest-low-usage-initialization.html

We have monitored file descriptor leaks on systems running Apache Drill and have come to believe (but not proven) that they are associated with Drill's ElasticSearch plugin, and that in turn makes use of Calcite's ElasticSearch adapter.

Issue Links

• links to: GitHub Pull Request #2698