[server] Last close ZookeeperClient to prevent exceptions (#2018)

Author: LiebingYu

fluss-server/src/main/java/org/apache/fluss/server/authorizer/DefaultAuthorizer.java

@@ -159,10 +159,6 @@ public void startup() throws Exception {
 
     @Override
     public void close() {
-        if (zooKeeperClient != null) {
-            zooKeeperClient.close();
-        }
-
         if (aclChangeNotificationWatcher != null) {
             aclChangeNotificationWatcher.stop();
         }

fluss-server/src/main/java/org/apache/fluss/server/authorizer/ZkNodeChangeNotificationWatcher.java

@@ -91,7 +91,7 @@ public void stop() {
             return;
         }
         running = false;
-        LOG.info("Stopping TableChangeWatcher");
+        LOG.info("Stopping ZkNodeChangeNotificationWatcher");
         curatorCache.close();
     }
 

fluss-server/src/main/java/org/apache/fluss/server/tablet/TabletServer.java

@@ -382,10 +382,6 @@ CompletableFuture<Void> stopServices() {
             }
 
             try {
-                if (zkClient != null) {
-                    zkClient.close();
-                }
-
                 // TODO currently, rpc client don't have timeout logic. After implementing the
                 // timeout logic, we need to move the closure of rpc client to after the closure of
                 // replica manager.
@@ -431,6 +427,9 @@ CompletableFuture<Void> stopServices() {
                     lakeCatalogDynamicLoader.close();
                 }
 
+                if (zkClient != null) {
+                    zkClient.close();
+                }
             } catch (Throwable t) {
                 exception = ExceptionUtils.firstOrSuppressed(t, exception);
             }