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/fury.git


The following commit(s) were added to refs/heads/main by this push:
     new fd85941f fix(kotlin): fix kotlin compile error (#2173)
fd85941f is described below

commit fd85941f0d16dd13a30daddd1f634ab6eec1795a
Author: Shawn Yang <[email protected]>
AuthorDate: Thu Apr 24 01:36:15 2025 +0800

    fix(kotlin): fix kotlin compile error (#2173)
    
    <!--
    **Thanks for contributing to Fury.**
    
    **If this is your first time opening a PR on fury, you can refer to
    
[CONTRIBUTING.md](https://github.com/apache/fury/blob/main/CONTRIBUTING.md).**
    
    Contribution Checklist
    
    - The **Apache Fury (incubating)** community has restrictions on the
    naming of pr titles. You can also find instructions in
    [CONTRIBUTING.md](https://github.com/apache/fury/blob/main/CONTRIBUTING.md).
    
    - Fury 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.
    -->
    
    ## What does this PR do?
    
    <!-- Describe the purpose of this PR. -->
    
    ## 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.
    -->
---
 .../org/apache/fury/serializer/kotlin/UnsignedArraySerializers.kt    | 5 -----
 .../kotlin/org/apache/fury/serializer/kotlin/UnsignedSerializer.kt   | 5 -----
 2 files changed, 10 deletions(-)

diff --git 
a/kotlin/src/main/kotlin/org/apache/fury/serializer/kotlin/UnsignedArraySerializers.kt
 
b/kotlin/src/main/kotlin/org/apache/fury/serializer/kotlin/UnsignedArraySerializers.kt
index 5a4e55a4..374d0eea 100644
--- 
a/kotlin/src/main/kotlin/org/apache/fury/serializer/kotlin/UnsignedArraySerializers.kt
+++ 
b/kotlin/src/main/kotlin/org/apache/fury/serializer/kotlin/UnsignedArraySerializers.kt
@@ -24,7 +24,6 @@ package org.apache.fury.serializer.kotlin
 import org.apache.fury.Fury
 import org.apache.fury.memory.MemoryBuffer
 import org.apache.fury.serializer.Serializer
-import org.apache.fury.type.Type
 
 public abstract class AbstractDelegatingArraySerializer<T, T_Delegate>(
   fury: Fury,
@@ -39,10 +38,6 @@ public abstract class AbstractDelegatingArraySerializer<T, 
T_Delegate>(
 
   protected abstract fun fromDelegateClass(value: T_Delegate): T
 
-  override fun getXtypeId(): Short {
-    return (-Type.LIST.id).toShort()
-  }
-
   override fun xwrite(buffer: MemoryBuffer, value: T) {
     write(buffer, value)
   }
diff --git 
a/kotlin/src/main/kotlin/org/apache/fury/serializer/kotlin/UnsignedSerializer.kt
 
b/kotlin/src/main/kotlin/org/apache/fury/serializer/kotlin/UnsignedSerializer.kt
index 2e0df5c2..e2492c2a 100644
--- 
a/kotlin/src/main/kotlin/org/apache/fury/serializer/kotlin/UnsignedSerializer.kt
+++ 
b/kotlin/src/main/kotlin/org/apache/fury/serializer/kotlin/UnsignedSerializer.kt
@@ -22,7 +22,6 @@ package org.apache.fury.serializer.kotlin
 import org.apache.fury.Fury
 import org.apache.fury.memory.MemoryBuffer
 import org.apache.fury.serializer.Serializers
-import org.apache.fury.type.Type
 
 /**
  * UByteSerializer
@@ -35,7 +34,6 @@ public class UByteSerializer(
   Serializers.CrossLanguageCompatibleSerializer<UByte>(
     fury,
     UByte::class.java,
-    Type.UINT8.id,
     fury.isBasicTypesRefIgnored,
     true
   ) {
@@ -60,7 +58,6 @@ public class UShortSerializer(
   Serializers.CrossLanguageCompatibleSerializer<UShort>(
     fury,
     UShort::class.java,
-    Type.UINT16.id,
     fury.isBasicTypesRefIgnored,
     true
   ) {
@@ -84,7 +81,6 @@ public class UIntSerializer(
   Serializers.CrossLanguageCompatibleSerializer<UInt>(
     fury,
     UInt::class.java,
-    Type.UINT32.id,
     fury.isBasicTypesRefIgnored,
     true
   ) {
@@ -109,7 +105,6 @@ public class ULongSerializer(
   Serializers.CrossLanguageCompatibleSerializer<ULong>(
     fury,
     ULong::class.java,
-    Type.UINT64.id,
     fury.isBasicTypesRefIgnored,
     true
   ) {


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

Reply via email to