This is an automated email from the ASF dual-hosted git repository.
pandalee pushed a commit to branch releases-0.10
in repository https://gitbox.apache.org/repos/asf/fury.git
The following commit(s) were added to refs/heads/releases-0.10 by this push:
new 3f1f0071 fix(kotlin): fix kotlin compile error (#2173) (#2188)
3f1f0071 is described below
commit 3f1f00719f700fa1c889555ed2509ee040afe82c
Author: LiangliangSui <[email protected]>
AuthorDate: Thu Apr 24 22:35:22 2025 +0800
fix(kotlin): fix kotlin compile error (#2173) (#2188)
<!--
**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. -->
cherrypick https://github.com/apache/fury/pull/2173 into releases-0.10
## 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.
-->
<!--
**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.
-->
Co-authored-by: Shawn Yang <[email protected]>
---
.../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]
