This is an automated email from the ASF dual-hosted git repository.
suiliangliang pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/fury.git
The following commit(s) were added to refs/heads/main by this push:
new 7421d6ce chore(java): reduce java test log (#2217)
7421d6ce is described below
commit 7421d6ce4c8eba4dc084a1e422aee4ebe901a3ea
Author: Shawn Yang <shawn.ck.y...@gmail.com>
AuthorDate: Mon May 12 09:45:43 2025 +0800
chore(java): reduce java test log (#2217)
## What does this PR do?
reduce java test log
## Related issues
<!--
Is there any related issue? Please attach here.
- #xxxx0
- #xxxx1
- #xxxx2
-->
## Does this PR introduce any user-facing change?
<!--
If any user-facing interface changes, please [open an
issue](https://github.com/apache/fury/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.
-->
---
.../test/java/org/apache/fury/FuryTestBase.java | 19 +++++++-
.../CodegenCompatibleSerializerTest.java | 5 +-
.../fury/serializer/CodegenSerializerTest.java | 22 ++++-----
.../fury/serializer/MetaSharedCompatibleTest.java | 2 +-
.../serializer/MetaSharedObjectSerializerTest.java | 5 +-
.../NonexistentClassSerializersTest.java | 2 +-
.../serializer/ProtocolInteroperabilityTest.java | 8 ++--
.../serializer/collection/MapSerializersTest.java | 56 ++++++++--------------
8 files changed, 61 insertions(+), 58 deletions(-)
diff --git a/java/fury-core/src/test/java/org/apache/fury/FuryTestBase.java
b/java/fury-core/src/test/java/org/apache/fury/FuryTestBase.java
index 54584d22..5737d60c 100644
--- a/java/fury-core/src/test/java/org/apache/fury/FuryTestBase.java
+++ b/java/fury-core/src/test/java/org/apache/fury/FuryTestBase.java
@@ -51,14 +51,22 @@ import org.testng.annotations.DataProvider;
public abstract class FuryTestBase {
private static final ThreadLocal<Fury> javaFuryLocal =
ThreadLocal.withInitial(
- () ->
Fury.builder().withLanguage(Language.JAVA).requireClassRegistration(false).build());
+ () ->
+ Fury.builder()
+ .withLanguage(Language.JAVA)
+ .requireClassRegistration(false)
+ .suppressClassRegistrationWarnings(true)
+ .build());
public static Fury getJavaFury() {
return javaFuryLocal.get();
}
public static FuryBuilder builder() {
- return
Fury.builder().withLanguage(Language.JAVA).requireClassRegistration(false);
+ return Fury.builder()
+ .withLanguage(Language.JAVA)
+ .suppressClassRegistrationWarnings(true)
+ .requireClassRegistration(false);
}
@DataProvider
@@ -97,6 +105,7 @@ public abstract class FuryTestBase {
.withRefTracking(true)
.withCodegen(false)
.requireClassRegistration(false)
+ .suppressClassRegistrationWarnings(true)
.build()
},
{
@@ -105,6 +114,7 @@ public abstract class FuryTestBase {
.withRefTracking(false)
.withCodegen(false)
.requireClassRegistration(false)
+ .suppressClassRegistrationWarnings(true)
.build()
}
};
@@ -174,6 +184,7 @@ public abstract class FuryTestBase {
.withRefTracking(true)
.withCodegen(false)
.requireClassRegistration(false)
+ .suppressClassRegistrationWarnings(true)
.build()
},
{
@@ -182,6 +193,7 @@ public abstract class FuryTestBase {
.withRefTracking(false)
.withCodegen(false)
.requireClassRegistration(false)
+ .suppressClassRegistrationWarnings(true)
.build()
},
{
@@ -190,6 +202,7 @@ public abstract class FuryTestBase {
.withRefTracking(true)
.withCodegen(true)
.requireClassRegistration(false)
+ .suppressClassRegistrationWarnings(true)
.build()
},
{
@@ -198,6 +211,7 @@ public abstract class FuryTestBase {
.withRefTracking(false)
.withCodegen(true)
.requireClassRegistration(false)
+ .suppressClassRegistrationWarnings(true)
.build()
},
};
@@ -210,6 +224,7 @@ public abstract class FuryTestBase {
Fury.builder()
.withLanguage(Language.JAVA)
.requireClassRegistration(false)
+ .suppressClassRegistrationWarnings(true)
.withScopedMetaShare(false);
return new Object[][] {
{
diff --git
a/java/fury-core/src/test/java/org/apache/fury/serializer/CodegenCompatibleSerializerTest.java
b/java/fury-core/src/test/java/org/apache/fury/serializer/CodegenCompatibleSerializerTest.java
index e9cbf399..1fd2fe43 100644
---
a/java/fury-core/src/test/java/org/apache/fury/serializer/CodegenCompatibleSerializerTest.java
+++
b/java/fury-core/src/test/java/org/apache/fury/serializer/CodegenCompatibleSerializerTest.java
@@ -58,7 +58,10 @@ public class CodegenCompatibleSerializerTest extends
FuryTestBase {
}
private FuryBuilder furyBuilder() {
- return
Fury.builder().withLanguage(Language.JAVA).requireClassRegistration(false);
+ return Fury.builder()
+ .withLanguage(Language.JAVA)
+ .requireClassRegistration(false)
+ .suppressClassRegistrationWarnings(true);
}
@Test(dataProvider = "config")
diff --git
a/java/fury-core/src/test/java/org/apache/fury/serializer/CodegenSerializerTest.java
b/java/fury-core/src/test/java/org/apache/fury/serializer/CodegenSerializerTest.java
index 7da08486..9df16651 100644
---
a/java/fury-core/src/test/java/org/apache/fury/serializer/CodegenSerializerTest.java
+++
b/java/fury-core/src/test/java/org/apache/fury/serializer/CodegenSerializerTest.java
@@ -56,7 +56,7 @@ public class CodegenSerializerTest extends FuryTestBase {
@Test
public void testSimpleBean() {
- Fury fury =
Fury.builder().withLanguage(Language.JAVA).requireClassRegistration(false).build();
+ Fury fury =
builder().withLanguage(Language.JAVA).requireClassRegistration(false).build();
// serDe(fury, new A());
B b = new B();
b.f1 = "str1";
@@ -83,7 +83,7 @@ public class CodegenSerializerTest extends FuryTestBase {
public void testSerializeCircularReference() {
Cyclic cyclic = Cyclic.create(true);
Fury fury =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withRefTracking(true)
.requireClassRegistration(false)
@@ -129,7 +129,7 @@ public class CodegenSerializerTest extends FuryTestBase {
circular2.circular1 = circular1;
circular2.circular2 = circular2;
Fury fury =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withRefTracking(true)
.requireClassRegistration(false)
@@ -178,7 +178,7 @@ public class CodegenSerializerTest extends FuryTestBase {
@Test
public void testCacheNonFinalClassInfo() {
- Fury fury =
Fury.builder().withLanguage(Language.JAVA).requireClassRegistration(false).build();
+ Fury fury =
builder().withLanguage(Language.JAVA).requireClassRegistration(false).build();
TestCacheNonFinalClassInfo obj = new TestCacheNonFinalClassInfo();
obj.finalList = new ArrayList<>(ImmutableList.of("a", "b"));
obj.nonFinalPublicList =
@@ -213,7 +213,7 @@ public class CodegenSerializerTest extends FuryTestBase {
pojo.f7 = 2;
pojo.f8 = 2L;
int length =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withNumberCompressed(false)
.withCompatibleMode(CompatibleMode.SCHEMA_CONSISTENT)
@@ -222,7 +222,7 @@ public class CodegenSerializerTest extends FuryTestBase {
.serialize(pojo)
.length;
Fury fury =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withNumberCompressed(true)
.withCompatibleMode(CompatibleMode.SCHEMA_CONSISTENT)
@@ -232,7 +232,7 @@ public class CodegenSerializerTest extends FuryTestBase {
byte[] bytes = fury.serialize(pojo);
{
Fury fury1 =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withIntCompressed(true)
.withLongCompressed(false)
@@ -248,7 +248,7 @@ public class CodegenSerializerTest extends FuryTestBase {
}
{
Fury fury1 =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withIntCompressed(false)
.withLongCompressed(true)
@@ -286,7 +286,7 @@ public class CodegenSerializerTest extends FuryTestBase {
@Test
public void testFinalTypeField() {
Fury fury =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withRefTracking(false)
.requireClassRegistration(true)
@@ -318,7 +318,7 @@ public class CodegenSerializerTest extends FuryTestBase {
@Test
public void testAnonymousClass() {
- Fury fury = Fury.builder().requireClassRegistration(false).build();
+ Fury fury = builder().requireClassRegistration(false).build();
// anonymous class never static
serDeCheckSerializer(fury, subclassObject, "Serializer");
}
@@ -330,7 +330,7 @@ public class CodegenSerializerTest extends FuryTestBase {
}
A a = new A();
a.f1 = 10;
- Fury fury = Fury.builder().requireClassRegistration(false).build();
+ Fury fury = builder().requireClassRegistration(false).build();
A o = serDe(fury, a);
assertEquals(o.f1, a.f1);
// local class never static
diff --git
a/java/fury-core/src/test/java/org/apache/fury/serializer/MetaSharedCompatibleTest.java
b/java/fury-core/src/test/java/org/apache/fury/serializer/MetaSharedCompatibleTest.java
index 3d5b64eb..1ee8f1c0 100644
---
a/java/fury-core/src/test/java/org/apache/fury/serializer/MetaSharedCompatibleTest.java
+++
b/java/fury-core/src/test/java/org/apache/fury/serializer/MetaSharedCompatibleTest.java
@@ -100,7 +100,7 @@ public class MetaSharedCompatibleTest extends FuryTestBase {
}
private static FuryBuilder furyBuilder() {
- return Fury.builder()
+ return builder()
.withLanguage(Language.JAVA)
.withMetaShare(true)
.withCompatibleMode(CompatibleMode.COMPATIBLE)
diff --git
a/java/fury-core/src/test/java/org/apache/fury/serializer/MetaSharedObjectSerializerTest.java
b/java/fury-core/src/test/java/org/apache/fury/serializer/MetaSharedObjectSerializerTest.java
index d102c2cd..71db5a02 100644
---
a/java/fury-core/src/test/java/org/apache/fury/serializer/MetaSharedObjectSerializerTest.java
+++
b/java/fury-core/src/test/java/org/apache/fury/serializer/MetaSharedObjectSerializerTest.java
@@ -19,7 +19,6 @@
package org.apache.fury.serializer;
-import org.apache.fury.Fury;
import org.apache.fury.FuryTestBase;
import org.apache.fury.ThreadSafeFury;
import org.apache.fury.codegen.JaninoUtils;
@@ -50,7 +49,7 @@ public class MetaSharedObjectSerializerTest extends
FuryTestBase {
Class<?> cls1 = JaninoUtils.compileClass(getClass().getClassLoader(), "",
"TestA", codeA);
Class<?> cls2 = JaninoUtils.compileClass(getClass().getClassLoader(), "",
"TestA", codeB);
ThreadSafeFury fury1 =
- Fury.builder()
+ builder()
.withRefTracking(true)
.requireClassRegistration(false)
.withDeserializeNonexistentClass(true)
@@ -61,7 +60,7 @@ public class MetaSharedObjectSerializerTest extends
FuryTestBase {
.withClassLoader(cls1.getClassLoader())
.buildThreadSafeFury();
ThreadSafeFury fury2 =
- Fury.builder()
+ builder()
.withRefTracking(true)
.requireClassRegistration(false)
.withDeserializeNonexistentClass(true)
diff --git
a/java/fury-core/src/test/java/org/apache/fury/serializer/NonexistentClassSerializersTest.java
b/java/fury-core/src/test/java/org/apache/fury/serializer/NonexistentClassSerializersTest.java
index 05a5dcdc..ed323e50 100644
---
a/java/fury-core/src/test/java/org/apache/fury/serializer/NonexistentClassSerializersTest.java
+++
b/java/fury-core/src/test/java/org/apache/fury/serializer/NonexistentClassSerializersTest.java
@@ -68,7 +68,7 @@ public class NonexistentClassSerializersTest extends
FuryTestBase {
}
private FuryBuilder furyBuilder(boolean scoped) {
- return Fury.builder()
+ return builder()
.withLanguage(Language.JAVA)
.withCompatibleMode(CompatibleMode.COMPATIBLE)
.requireClassRegistration(false)
diff --git
a/java/fury-core/src/test/java/org/apache/fury/serializer/ProtocolInteroperabilityTest.java
b/java/fury-core/src/test/java/org/apache/fury/serializer/ProtocolInteroperabilityTest.java
index 38b0a114..f52580eb 100644
---
a/java/fury-core/src/test/java/org/apache/fury/serializer/ProtocolInteroperabilityTest.java
+++
b/java/fury-core/src/test/java/org/apache/fury/serializer/ProtocolInteroperabilityTest.java
@@ -68,7 +68,7 @@ public class ProtocolInteroperabilityTest extends
FuryTestBase {
.map(
c ->
new Object[] {
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withRefTracking((Boolean) c[0])
.withNumberCompressed((Boolean) c[1])
@@ -77,7 +77,7 @@ public class ProtocolInteroperabilityTest extends
FuryTestBase {
.withCompatibleMode((CompatibleMode) c[3])
.requireClassRegistration(false)
.build(),
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withRefTracking((Boolean) c[0])
.withNumberCompressed((Boolean) c[1])
@@ -236,7 +236,7 @@ public class ProtocolInteroperabilityTest extends
FuryTestBase {
.map(
c ->
new Object[] {
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withMetaShare(true)
.withCompatibleMode((CompatibleMode) c[0])
@@ -244,7 +244,7 @@ public class ProtocolInteroperabilityTest extends
FuryTestBase {
.withCodegen(false)
.requireClassRegistration(false)
.build(),
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withMetaShare(true)
.withCompatibleMode((CompatibleMode) c[0])
diff --git
a/java/fury-core/src/test/java/org/apache/fury/serializer/collection/MapSerializersTest.java
b/java/fury-core/src/test/java/org/apache/fury/serializer/collection/MapSerializersTest.java
index f0b94ddf..2e38f728 100644
---
a/java/fury-core/src/test/java/org/apache/fury/serializer/collection/MapSerializersTest.java
+++
b/java/fury-core/src/test/java/org/apache/fury/serializer/collection/MapSerializersTest.java
@@ -71,7 +71,7 @@ public class MapSerializersTest extends FuryTestBase {
boolean scopedMetaShare,
CompatibleMode compatibleMode) {
Fury fury =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withRefTracking(trackingRef)
.requireClassRegistration(false)
@@ -195,7 +195,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test(dataProvider = "referenceTrackingConfig")
public void testBasicMap(boolean referenceTrackingConfig) {
Fury fury =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withRefTracking(referenceTrackingConfig)
.requireClassRegistration(false)
@@ -227,7 +227,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test(dataProvider = "referenceTrackingConfig")
public void testBasicMapNested(boolean referenceTrackingConfig) {
Fury fury =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withRefTracking(referenceTrackingConfig)
.requireClassRegistration(false)
@@ -249,7 +249,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test(dataProvider = "referenceTrackingConfig")
public void testMapGenerics(boolean referenceTrackingConfig) {
Fury fury =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withRefTracking(referenceTrackingConfig)
.requireClassRegistration(false)
@@ -266,7 +266,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test(dataProvider = "referenceTrackingConfig")
public void testSortedMap(boolean referenceTrackingConfig) {
Fury fury =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withRefTracking(referenceTrackingConfig)
.requireClassRegistration(false)
@@ -301,7 +301,7 @@ public class MapSerializersTest extends FuryTestBase {
public void testTreeMap() {
boolean referenceTracking = true;
Fury fury =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withRefTracking(referenceTracking)
.requireClassRegistration(false)
@@ -413,7 +413,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test
public void testNoArgConstructor() {
- Fury fury =
Fury.builder().withLanguage(Language.JAVA).requireClassRegistration(false).build();
+ Fury fury =
builder().withLanguage(Language.JAVA).requireClassRegistration(false).build();
Map<String, Integer> map = newInnerMap();
Assert.assertEquals(jdkDeserialize(jdkSerialize(map)), map);
serDeCheck(fury, map);
@@ -427,7 +427,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test
public void testMapNoJIT() {
- Fury fury =
Fury.builder().withLanguage(Language.JAVA).withCodegen(false).build();
+ Fury fury =
builder().withLanguage(Language.JAVA).withCodegen(false).build();
serDeCheck(fury, new HashMap<>(ImmutableMap.of("a", 1, "b", 2)));
serDeCheck(fury, new HashMap<>(ImmutableMap.of("a", "v1", "b", "v2")));
serDeCheck(fury, new HashMap<>(ImmutableMap.of(1, 2, 3, 4)));
@@ -532,7 +532,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test(dataProvider = "enableCodegen")
public void testDefaultMapSerializer(boolean enableCodegen) {
Fury fury =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.withCodegen(enableCodegen)
.requireClassRegistration(false)
@@ -586,13 +586,13 @@ public class MapSerializersTest extends FuryTestBase {
@Test
public void testGenericMapBound() {
Fury fury1 =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.requireClassRegistration(false)
.withCodegen(false)
.build();
Fury fury2 =
- Fury.builder()
+ builder()
.withLanguage(Language.JAVA)
.requireClassRegistration(false)
.withCodegen(false)
@@ -775,11 +775,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test
public void testMapChunkRefTracking() {
Fury fury =
- Fury.builder()
- .withRefTracking(true)
- .withCodegen(false)
- .requireClassRegistration(false)
- .build();
+
builder().withRefTracking(true).withCodegen(false).requireClassRegistration(false).build();
Map<String, Integer> map = new HashMap<>();
for (int i = 0; i < 1; i++) {
map.put("k" + i, i);
@@ -791,11 +787,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test
public void testMapChunkRefTrackingGenerics() {
Fury fury =
- Fury.builder()
- .withRefTracking(true)
- .withCodegen(false)
- .requireClassRegistration(false)
- .build();
+
builder().withRefTracking(true).withCodegen(false).requireClassRegistration(false).build();
MapFields obj = new MapFields();
Map<String, Integer> map = new HashMap<>();
@@ -810,7 +802,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test(dataProvider = "referenceTrackingConfig")
public void testMapFieldsChunkSerializer(boolean referenceTrackingConfig) {
Fury fury =
- Fury.builder()
+ builder()
.withRefTracking(referenceTrackingConfig)
.withCodegen(false)
.requireClassRegistration(false)
@@ -849,7 +841,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test(dataProvider = "referenceTrackingConfig")
public void testMapFieldStructCodegen1(boolean referenceTrackingConfig) {
Fury fury =
- Fury.builder()
+ builder()
.withRefTracking(referenceTrackingConfig)
.withCodegen(true)
.requireClassRegistration(false)
@@ -870,7 +862,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test(dataProvider = "referenceTrackingConfig")
public void testMapFieldStructCodegen2(boolean referenceTrackingConfig) {
Fury fury =
- Fury.builder()
+ builder()
.withRefTracking(referenceTrackingConfig)
.withCodegen(true)
.requireClassRegistration(false)
@@ -892,7 +884,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test(dataProvider = "referenceTrackingConfig")
public void testMapFieldStructCodegen3(boolean referenceTrackingConfig) {
Fury fury =
- Fury.builder()
+ builder()
.withRefTracking(referenceTrackingConfig)
.withCodegen(true)
.requireClassRegistration(false)
@@ -916,7 +908,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test(dataProvider = "referenceTrackingConfig")
public void testNestedMapFieldStructCodegen(boolean referenceTrackingConfig)
{
Fury fury =
- Fury.builder()
+ builder()
.withRefTracking(referenceTrackingConfig)
.withCodegen(true)
.requireClassRegistration(false)
@@ -958,10 +950,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test(dataProvider = "referenceTrackingConfig")
public void testWildcard(boolean referenceTrackingConfig) {
Fury fury =
- Fury.builder()
- .withRefTracking(referenceTrackingConfig)
- .requireClassRegistration(false)
- .build();
+
builder().withRefTracking(referenceTrackingConfig).requireClassRegistration(false).build();
MapWildcardFieldStruct1 struct = new MapWildcardFieldStruct1();
struct.f0 = ofHashMap("k", 1);
struct.f1 = ofHashMap("k1", new Wildcard<>());
@@ -982,10 +971,7 @@ public class MapSerializersTest extends FuryTestBase {
@Test(dataProvider = "referenceTrackingConfig")
public void testNestedListMap(boolean referenceTrackingConfig) {
Fury fury =
- Fury.builder()
- .withRefTracking(referenceTrackingConfig)
- .requireClassRegistration(false)
- .build();
+
builder().withRefTracking(referenceTrackingConfig).requireClassRegistration(false).build();
NestedListMap o = new NestedListMap();
o.map1 = ofArrayList(ofHashMap("k1", "v"));
o.map2 = ofArrayList(ofHashMap("k2", "2"));
@@ -1002,7 +988,7 @@ public class MapSerializersTest extends FuryTestBase {
NestedMapCollectionGenericTestClass obj = new
NestedMapCollectionGenericTestClass();
obj.map = new LinkedHashMap<>();
obj.map.put("obj", ofHashMap("obj", 1, "b", ofArrayList(10)));
- Fury fury =
Fury.builder().requireClassRegistration(false).withCodegen(enableCodegen).build();
+ Fury fury =
builder().requireClassRegistration(false).withCodegen(enableCodegen).build();
fury.deserialize(fury.serialize(obj));
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@fury.apache.org
For additional commands, e-mail: commits-h...@fury.apache.org
