Skip to content

feat(rust): support thread safe Arc<dyn Any + Send + Sync> type#3736

Merged
chaokunyang merged 2 commits into
apache:mainfrom
chaokunyang:fix_unknown_case_threadsafe
Jun 3, 2026
Merged

feat(rust): support thread safe Arc<dyn Any + Send + Sync> type#3736
chaokunyang merged 2 commits into
apache:mainfrom
chaokunyang:fix_unknown_case_threadsafe

Conversation

@chaokunyang
Copy link
Copy Markdown
Collaborator

@chaokunyang chaokunyang commented Jun 3, 2026

Why?

What does this PR do?

Related issues

Closes #3732

AI Contribution Checklist

  • Substantial AI assistance was used in this PR: yes / no
  • If yes, I included a completed AI Contribution Checklist in this PR description and the required AI Usage Disclosure.
  • If yes, my PR description includes the required ai_review summary and screenshot evidence of the final clean AI review results from both fresh reviewers on the current PR diff or current HEAD after the latest code changes.

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 requested review from pandalee99 and urlyy June 3, 2026 16:33
@chaokunyang chaokunyang merged commit e637cf7 into apache:main Jun 3, 2026
47 checks passed
chaokunyang added a commit that referenced this pull request Jun 4, 2026
## Why?



## What does this PR do?



## Related issues

#3732 
#3736

## AI Contribution Checklist



- [ ] Substantial AI assistance was used in this PR: `yes` / `no`
- [ ] If `yes`, I included a completed [AI Contribution
Checklist](https://github.com/apache/fory/blob/main/AI_POLICY.md#9-contributor-checklist-for-ai-assisted-prs)
in this PR description and the required `AI Usage Disclosure`.
- [ ] If `yes`, my PR description includes the required `ai_review`
summary and screenshot evidence of the final clean AI review results
from both fresh reviewers on the current PR diff or current HEAD after
the latest code changes.



## 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
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