Copilot commented on code in PR #15651:
URL: https://github.com/apache/dubbo/pull/15651#discussion_r2292822094
##########
dubbo-common/src/main/java/org/apache/dubbo/common/extension/support/WrapperComparator.java:
##########
@@ -25,7 +25,7 @@
import java.util.Comparator;
/**
- * OrderComparator
+ * WrapperComparator
* Derived from {@link ActivateComparator}
Review Comment:
The JavaDoc comment should include a brief description of what the
WrapperComparator does, not just the class name. Consider adding a sentence
explaining its purpose and how it differs from or relates to the
ActivateComparator it derives from.
```suggestion
* A comparator for ordering wrapper extension classes based on their {@code
@Activate} or {@code @Wrapper} annotation order values.
* <p>
* This comparator is similar to {@link ActivateComparator}, but is
specifically used for wrapper classes that may be annotated with {@code
@Wrapper}.
* It ensures that wrappers are sorted according to their specified order,
supporting both legacy and new annotation styles.
```
##########
dubbo-common/src/main/java/org/apache/dubbo/common/extension/support/ActivateComparator.java:
##########
@@ -33,7 +33,7 @@
import java.util.concurrent.ConcurrentHashMap;
/**
- * OrderComparator
+ * ActivateComparator
Review Comment:
The JavaDoc comment should include a brief description of what the
ActivateComparator does, not just the class name. Consider adding a sentence
explaining its purpose in the Dubbo extension system.
```suggestion
* A comparator for ordering Dubbo extension classes annotated with {@link
org.apache.dubbo.common.extension.Activate}.
* <p>
* This comparator determines the activation order of extensions based on
their {@code order}, {@code before}, and {@code after}
* attributes in the Dubbo extension system. It is used to ensure that
extensions are activated in the correct sequence
* according to their dependencies and priorities.
```
--
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]
