(iceberg) branch main updated: REST: Fix spurious warning when shutting down refresh executor (#10087)

[amoghj]

This is an automated email from the ASF dual-hosted git repository.

amoghj pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg.git


The following commit(s) were added to refs/heads/main by this push:
     new 356c6cd308 REST: Fix spurious warning when shutting down refresh 
executor (#10087)
356c6cd308 is described below

commit 356c6cd3089278349f0bd0ad6a12ea03a291a722
Author: Alexandre Dutra <adu...@users.noreply.github.com>
AuthorDate: Thu Apr 4 21:36:24 2024 +0200

    REST: Fix spurious warning when shutting down refresh executor (#10087)
---
 core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java 
b/core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java
index 96aa14b128..f8a47f19fd 100644
--- a/core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java
+++ b/core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java
@@ -602,7 +602,7 @@ public class RESTSessionCatalog extends 
BaseViewSessionCatalog
           });
 
       try {
-        if (service.awaitTermination(1, TimeUnit.MINUTES)) {
+        if (!service.awaitTermination(1, TimeUnit.MINUTES)) {
           LOG.warn("Timed out waiting for refresh executor to terminate");
         }
       } catch (InterruptedException e) {