swuferhong commented on code in PR #1380:
URL: https://github.com/apache/fluss/pull/1380#discussion_r2453810879
##########
fluss-client/src/main/java/org/apache/fluss/client/admin/Admin.java:
##########
@@ -492,4 +503,90 @@ ListOffsetsResult listOffsets(
* @return A CompletableFuture indicating completion of the operation.
*/
CompletableFuture<Void> alterClusterConfigs(Collection<AlterConfig>
configs);
+
+ /**
+ * Add server tag to the specified tabletServers, one tabletServer can
only have one serverTag.
+ *
+ * <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 ServerNotExistException} If the tabletServer in {@code
tabletServers} does not
+ * exist.
+ * <li>{@link ServerTagAlreadyExistException} If the server tag already
exists when {@code
+ * overWriteIfExists} is false.
+ * </ul>
+ *
+ * @param tabletServers the tabletServers we want to add server tags.
+ * @param serverTag the server tag to be added.
+ */
+ CompletableFuture<Void> addServerTag(List<Integer> tabletServers,
ServerTag serverTag);
Review Comment:
I will change the description of this api.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
