From 5bbcdd77b48561e252c31e9e8b731fbbdf5a7678 Mon Sep 17 00:00:00 2001 From: Karol Sobczak Date: Fri, 28 Jan 2022 00:44:50 +0100 Subject: [PATCH] Run HttpRemoteTask#sendUpdate in separate thread Sending update is potentially expensive operation therefore it should be run from separate thread. This fixes bug introduced by https://github.com/trinodb/trino/commit/613bd2f3feb42cc4aedf637260a2bc6672f323d4 --- .../main/java/io/trino/server/remotetask/HttpRemoteTask.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/trino-main/src/main/java/io/trino/server/remotetask/HttpRemoteTask.java b/core/trino-main/src/main/java/io/trino/server/remotetask/HttpRemoteTask.java index af31a9b89cb7..7ff03d397776 100644 --- a/core/trino-main/src/main/java/io/trino/server/remotetask/HttpRemoteTask.java +++ b/core/trino-main/src/main/java/io/trino/server/remotetask/HttpRemoteTask.java @@ -578,7 +578,7 @@ private synchronized void triggerUpdate() { // synchronized so that needsUpdate is not cleared in sendUpdate before actual request is sent needsUpdate.set(true); - sendUpdate(); + scheduleUpdate(); } private synchronized void sendUpdate()