fury icon indicating copy to clipboard operation
fury copied to clipboard
verified publisher icon apache

feat(java): Support copy capabilities for some classes without no-argument constructors

Open zhaommmmomo opened this issue 1 year ago • 1 comments

What does this PR do?

Some classes with no-argument constructors will report an error when calling copy().

This pr:

  • implement the copy method for the no-argument constructor serializer
  • add test case

Related issues

https://github.com/apache/fury/issues/1777 https://github.com/apache/fury/issues/1679

Does this PR introduce any user-facing change?

  • [ ] Does this PR introduce any public API change?
  • [ ] Does this PR introduce any binary protocol compatibility change?

Benchmark

zhaommmmomo avatar Aug 07 '24 12:08 zhaommmmomo

Very nice work, thank you @zhaommmmomo

chaokunyang avatar Aug 08 '24 15:08 chaokunyang

LGTM, this is not an easy work, thanks for your contribution @zhaommmmomo

This is what I should do

zhaommmmomo avatar Aug 13 '24 06:08 zhaommmmomo