This is an automated email from the ASF dual-hosted git repository.
chaokunyang pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/fory.git
The following commit(s) were added to refs/heads/main by this push:
new a5102c58 chore: refine validate serializers message for map/collection
(#2355)
a5102c58 is described below
commit a5102c58c1752aa6cb785aba46204a8f9cc749b1
Author: Shawn Yang <shawn.ck.y...@gmail.com>
AuthorDate: Thu Jun 19 11:48:51 2025 +0800
chore: refine validate serializers message for map/collection (#2355)
## What does this PR do?
<!-- Describe the purpose of this PR. -->
## Related issues
Closes #2345
## Does this PR introduce any user-facing change?
<!--
If any user-facing interface changes, please [open an
issue](https://github.com/apache/fory/issues/new/choose) describing the
need to do so and update the document if necessary.
-->
- [ ] Does this PR introduce any public API change?
- [ ] Does this PR introduce any binary protocol compatibility change?
## Benchmark
<!--
When the PR has an impact on performance (if you don't know whether the
PR will have an impact on performance, you can submit the PR first, and
if it will have impact on performance, the code reviewer will explain
it), be sure to attach a benchmark data here.
-->
---
.../main/java/org/apache/fory/serializer/SerializationUtils.java | 8 ++++----
.../test/java/javax/fory/test/ResolverValidateSerializerTest.java | 8 ++++----
2 files changed, 8 insertions(+), 8 deletions(-)
diff --git
a/java/fory-core/src/main/java/org/apache/fory/serializer/SerializationUtils.java
b/java/fory-core/src/main/java/org/apache/fory/serializer/SerializationUtils.java
index cbf80314..60f1ae47 100644
---
a/java/fory-core/src/main/java/org/apache/fory/serializer/SerializationUtils.java
+++
b/java/fory-core/src/main/java/org/apache/fory/serializer/SerializationUtils.java
@@ -25,8 +25,8 @@ import org.apache.fory.Fory;
import org.apache.fory.annotation.Internal;
import org.apache.fory.resolver.ClassInfo;
import org.apache.fory.resolver.TypeResolver;
-import org.apache.fory.serializer.collection.AbstractCollectionSerializer;
-import org.apache.fory.serializer.collection.AbstractMapSerializer;
+import org.apache.fory.serializer.collection.CollectionSerializer;
+import org.apache.fory.serializer.collection.MapSerializer;
@Internal
public class SerializationUtils {
@@ -59,7 +59,7 @@ public class SerializationUtils {
}
public static void validate(Class<?> type, Class<? extends Serializer>
serializerClass) {
- validateSerializer(type, serializerClass, Collection.class,
AbstractCollectionSerializer.class);
- validateSerializer(type, serializerClass, Map.class,
AbstractMapSerializer.class);
+ validateSerializer(type, serializerClass, Collection.class,
CollectionSerializer.class);
+ validateSerializer(type, serializerClass, Map.class, MapSerializer.class);
}
}
diff --git
a/java/fory-core/src/test/java/javax/fory/test/ResolverValidateSerializerTest.java
b/java/fory-core/src/test/java/javax/fory/test/ResolverValidateSerializerTest.java
index 546a9466..89320d72 100644
---
a/java/fory-core/src/test/java/javax/fory/test/ResolverValidateSerializerTest.java
+++
b/java/fory-core/src/test/java/javax/fory/test/ResolverValidateSerializerTest.java
@@ -31,8 +31,8 @@ import java.util.Set;
import org.apache.fory.Fory;
import org.apache.fory.memory.MemoryBuffer;
import org.apache.fory.serializer.Serializer;
-import org.apache.fory.serializer.collection.AbstractCollectionSerializer;
-import org.apache.fory.serializer.collection.AbstractMapSerializer;
+import org.apache.fory.serializer.collection.CollectionSerializer;
+import org.apache.fory.serializer.collection.MapSerializer;
import org.testng.annotations.Test;
public class ResolverValidateSerializerTest {
@@ -75,7 +75,7 @@ public class ResolverValidateSerializerTest {
return 0;
}
- public static final class ValidListSerializer extends
AbstractCollectionSerializer<ValidList> {
+ public static final class ValidListSerializer extends
CollectionSerializer<ValidList> {
public ValidListSerializer(Fory fory) {
super(fory, ValidList.class);
}
@@ -126,7 +126,7 @@ public class ResolverValidateSerializerTest {
return Collections.emptySet();
}
- public static final class ValidMapSerializer extends
AbstractMapSerializer<ValidMap> {
+ public static final class ValidMapSerializer extends
MapSerializer<ValidMap> {
public ValidMapSerializer(Fory fory) {
super(fory, ValidMap.class);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@fory.apache.org
For additional commands, e-mail: commits-h...@fory.apache.org
