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 cd018c910 chore(java): Remove print property names (#2860)
cd018c910 is described below

commit cd018c9101595347b9dd2bc1c0138ca2934d6fb6
Author: Yongha Kwon <[email protected]>
AuthorDate: Sat Nov 1 14:15:33 2025 +0900

    chore(java): Remove print property names (#2860)
    
    Removed debug print statement for descriptors.
    
    <!--
    **Thanks for contributing to Apache Fory™.**
    
    **If this is your first time opening a PR on fory, you can refer to
    
[CONTRIBUTING.md](https://github.com/apache/fory/blob/main/CONTRIBUTING.md).**
    
    Contribution Checklist
    
    - The **Apache Fory™** community has requirements on the naming of pr
    titles. You can also find instructions in
    [CONTRIBUTING.md](https://github.com/apache/fory/blob/main/CONTRIBUTING.md).
    
    - Apache Fory™ has a strong focus on performance. If the PR you submit
    will have an impact on performance, please benchmark it first and
    provide the benchmark result here.
    -->
    
    ## Why?
    
    Remove print property naems
    
    <!-- Describe the purpose of this PR. -->
    
    ## What does this PR do?
    
    Remove System.out.println.
    
    <!-- Describe the details of this PR. -->
    
    ## Related issues
    
    #2854
    
    <!--
    Is there any related issue? If this PR closes them you say say
    fix/closes:
    
    - #xxxx0
    - #xxxx1
    - Fixes #xxxx2
    -->
    
    ## 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.
    
    Delete section if not applicable.
    -->
    
    - [ ] 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.
    
    Delete section if not applicable.
    -->
    
    ---------
    
    Co-authored-by: Shawn Yang <[email protected]>
---
 .../src/main/java/org/apache/fory/serializer/ObjectSerializer.java       | 1 -
 1 file changed, 1 deletion(-)

diff --git 
a/java/fory-core/src/main/java/org/apache/fory/serializer/ObjectSerializer.java 
b/java/fory-core/src/main/java/org/apache/fory/serializer/ObjectSerializer.java
index e44d00744..b232e2be7 100644
--- 
a/java/fory-core/src/main/java/org/apache/fory/serializer/ObjectSerializer.java
+++ 
b/java/fory-core/src/main/java/org/apache/fory/serializer/ObjectSerializer.java
@@ -105,7 +105,6 @@ public final class ObjectSerializer<T> extends 
AbstractObjectSerializer<T> {
     }
     DescriptorGrouper grouper = 
typeResolver.createDescriptorGrouper(descriptors, false);
     descriptors = grouper.getSortedDescriptors();
-    
System.out.println(descriptors.stream().map(Descriptor::getName).collect(Collectors.toList()));
     if (isRecord) {
       List<String> fieldNames =
           
descriptors.stream().map(Descriptor::getName).collect(Collectors.toList());


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to