Re: [PR] optimize: replace fury with fory [incubator-seata]

["funky-eyes (via GitHub)" <gi...@apache.org>]

funky-eyes commented on code in PR #7885:
URL: https://github.com/apache/incubator-seata/pull/7885#discussion_r2638472703


##########
core/src/main/java/org/apache/seata/core/serializer/SerializerServiceLoader.java:
##########
@@ -53,8 +54,14 @@ public final class SerializerServiceLoader {
 
     private static final Map<String, Serializer> SERIALIZER_MAP = new HashMap<>();
 
+    private static final Map<String, String> SERIALIZER_ALIAS_MAP = new HashMap<>();
+
     private static final String SPLIT_CHAR = ",";
 
+    static {
+        SERIALIZER_ALIAS_MAP.put("fury", "fory");

Review Comment:
   Should we add aliases to the @LoadLevel annotation to better support compatibility when third-party components are renamed?



-- 
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: notifications-unsubscribe@seata.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@seata.apache.org
For additional commands, e-mail: notifications-help@seata.apache.org