Skip to content

Conversation

@chaokunyang
Copy link
Collaborator

@chaokunyang chaokunyang commented Sep 9, 2025

Why?

implement meta share mode for pyfory, so the struct can add/delete fields and have inconsistent schema bettween serialization and deserialization

What does this PR do?

Related issues

#2509
#1938
#2160
#2278

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

@chaokunyang chaokunyang changed the title feat(python): implement meta share mode for pyfory feat(python): implement meta share mode for pyfory compatible serialization Sep 9, 2025
@chaokunyang chaokunyang changed the title feat(python): implement meta share mode for pyfory compatible serialization feat(python): meta share mode for pyfory compatible serialization Sep 9, 2025
Copy link
Contributor

@pandalee99 pandalee99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@chaokunyang chaokunyang merged commit 3141860 into apache:main Sep 9, 2025
103 of 109 checks passed
chaokunyang added a commit that referenced this pull request Sep 10, 2025
#2602)

## Why?

align java and python compatible mode serialization

## What does this PR do?

<!-- Describe the details of this PR. -->

## Related issues

#2509
#1938
#2160
#2278
#2593

## 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.
-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants