Close listener refs in finally block (#108830) (#108837)

We will need to acquire a new listener in the catch block; therefore, we 
should not release the listenerRefs in the try-with-resources block,
which is executed before the catch block.

Relates #108580

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plugin/ComputeService.java

@@ -697,7 +697,7 @@ private void runComputeOnDataNode(
  final RefCountingListener listenerRefs = new RefCountingListener(
  ActionListener.runBefore(listener.map(unused -> new ComputeResponse(collectedProfiles)), responseHeadersCollector::finish)
  );
- try (listenerRefs) {
+ try {
  final AtomicBoolean cancelled = new AtomicBoolean();
  // run compute with target shards
  var internalSink = exchangeService.createSinkHandler(request.sessionId(), request.pragmas().exchangeBufferSize());
@@ -746,6 +746,8 @@ private void runComputeOnDataNode(
  exchangeService.finishSinkHandler(externalId, e);
  exchangeService.finishSinkHandler(request.sessionId(), e);
  listenerRefs.acquire().onFailure(e);
+ } finally {
+ listenerRefs.close();
  }
  }