deardeng commented on code in PR #42401:
URL: https://github.com/apache/doris/pull/42401#discussion_r1822589576
##########
fe/fe-core/src/main/java/org/apache/doris/alter/SystemHandler.java:
##########
@@ -193,19 +209,115 @@ public synchronized void process(String rawSql,
List<AlterClause> alterClauses,
/*
* check if the specified backends can be dropped
* 1. backend does not have any tablet.
- * 2. all tablets in backend have been recycled.
+ * 2. all tablets in backend have been recycled or leaky.
+ *
+ * sampleTablets: sample normal tablets
+ * sampleLeakyTablets: sample leaky tablets
+ *
*/
Review Comment:
这个函数加些注释吧，里面实现太复杂了
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
