address gujian's comments

Author: swuferhong

fluss-client/src/main/java/org/apache/fluss/client/admin/Admin.java

@@ -510,12 +510,12 @@ ListOffsetsResult listOffsets(
      * <p>If one tabletServer failed adding tag, none of the tags will take effect.
      *
      * <ul>
-     *   <li>{@link AuthorizationException} If the authenticated user doesn't have reset config
-     *       access to the cluster.
+     *   <li>{@link AuthorizationException} If the authenticated user doesn't have cluster
+     *       permissions.
      *   <li>{@link ServerNotExistException} If the tabletServer in {@code tabletServers} does not
      *       exist.
-     *   <li>{@link ServerTagAlreadyExistException} If the server tag already exists when {@code
-     *       overWriteIfExists} is false.
+     *   <li>{@link ServerTagAlreadyExistException} If the server tag already exists for any one of
+     *       the tabletServers.
      * </ul>
      *
      * @param tabletServers the tabletServers we want to add server tags.
@@ -529,12 +529,12 @@ ListOffsetsResult listOffsets(
      * <p>If one tabletServer failed removing tag, none of the tags will be removed.
      *
      * <ul>
-     *   <li>{@link AuthorizationException} If the authenticated user doesn't have reset config
-     *       access to the cluster.
+     *   <li>{@link AuthorizationException} If the authenticated user doesn't have cluster
+     *       permissions.
      *   <li>{@link ServerNotExistException} If the tabletServer in {@code tabletServers} does not
      *       exist.
-     *   <li>{@link ServerTagNotExistException} If the server tag does not exist when {@code
-     *       overWriteIfExists} is false.
+     *   <li>{@link ServerTagNotExistException} If the server tag does not exist for any one of the
+     *       tabletServers.
      * </ul>
      *
      * @param tabletServers the tabletServers we want to remove server tags.
@@ -552,8 +552,8 @@ ListOffsetsResult listOffsets(
      * rebalance task exists, an {@link RebalanceFailureException} will be thrown.
      *
      * <ul>
-     *   <li>{@link AuthorizationException} If the authenticated user doesn't have reset config
-     *       access to the cluster.
+     *   <li>{@link AuthorizationException} If the authenticated user doesn't have cluster
+     *       permissions.
      *   <li>{@link RebalanceFailureException} If the rebalance failed. Such as there is an ongoing
      *       execution.
      * </ul>
@@ -570,8 +570,8 @@ CompletableFuture<Map<TableBucket, RebalancePlanForBucket>> rebalance(
      * List the rebalance process.
      *
      * <ul>
-     *   <li>{@link AuthorizationException} If the authenticated user doesn't have reset config
-     *       access to the cluster.
+     *   <li>{@link AuthorizationException} If the authenticated user doesn't have cluster
+     *       permissions.
      *   <li>{@link NoRebalanceInProgressException} If there are no rebalance tasks in progress.
      * </ul>
      *
@@ -583,8 +583,8 @@ CompletableFuture<Map<TableBucket, RebalancePlanForBucket>> rebalance(
      * Cannel the rebalance task.
      *
      * <ul>
-     *   <li>{@link AuthorizationException} If the authenticated user doesn't have reset config
-     *       access to the cluster.
+     *   <li>{@link AuthorizationException} If the authenticated user doesn't have cluster
+     *       permissions.
      *   <li>{@link NoRebalanceInProgressException} If there are no rebalance tasks in progress.
      * </ul>
      */

fluss-rpc/src/main/java/org/apache/fluss/rpc/protocol/Errors.java

@@ -69,6 +69,7 @@
 import org.apache.fluss.exception.SecurityTokenException;
 import org.apache.fluss.exception.ServerNotExistException;
 import org.apache.fluss.exception.ServerTagAlreadyExistException;
+import org.apache.fluss.exception.ServerTagNotExistException;
 import org.apache.fluss.exception.StorageException;
 import org.apache.fluss.exception.TableAlreadyExistException;
 import org.apache.fluss.exception.TableNotExistException;
@@ -236,8 +237,7 @@ public enum Errors {
     SERVER_NOT_EXIST_EXCEPTION(58, "The server is not exist.", ServerNotExistException::new),
     SEVER_TAG_ALREADY_EXIST_EXCEPTION(
             59, "The server tag already exist.", ServerTagAlreadyExistException::new),
-    SEVER_TAG_NOT_EXIST_EXCEPTION(
-            60, "The server tag not exist.", ServerTagAlreadyExistException::new),
+    SEVER_TAG_NOT_EXIST_EXCEPTION(60, "The server tag not exist.", ServerTagNotExistException::new),
     REBALANCE_FAILURE_EXCEPTION(61, "The rebalance task failure.", RebalanceFailureException::new),
     NO_REBALANCE_IN_PROGRESS_EXCEPTION(
             62, "No rebalance task in progress.", NoRebalanceInProgressException::new);