swuferhong commented on code in PR #1452:
URL: https://github.com/apache/fluss/pull/1452#discussion_r2664930273


##########
fluss-client/src/main/java/org/apache/fluss/client/admin/Admin.java:
##########
@@ -588,6 +594,9 @@ ListOffsetsResult listOffsets(
      *       permissions.
      *   <li>{@link NoRebalanceInProgressException} If there are no rebalance 
tasks in progress.
      * </ul>
+     *
+     * @param rebalanceId the rebalance id to cancel, if it is null means 
cancel the latest
+     *     rebalance task. If rebalanceId is not exists in server, nothing 
will be done.

Review Comment:
   I change it to throw exception.



-- 
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]

Reply via email to