Skip to content

[JavaScript] support polymorphism#814

Merged
theweipeng merged 1 commit intoapache:mainfrom
theweipeng:issue_813
Aug 1, 2023
Merged

[JavaScript] support polymorphism#814
theweipeng merged 1 commit intoapache:mainfrom
theweipeng:issue_813

Conversation

@theweipeng
Copy link
Copy Markdown
Member

@theweipeng theweipeng commented Aug 1, 2023

What do these changes do?

  • Add a tool function called Type.Any which would use the anySerializer during compile time.
  • Fixed a bug where the dependencies serializer was null when build anySerializer, so we can't init the dependencies serializer when build it. We replaced it by getting the serializer by ID or tag every time we read or write. However, this modification could hurt the performance of Type.Any.

Related issue number

Closes #813

Check code requirements

  • tests added / passed (if needed)
  • Ensure all linting tests pass, see here for how to run them

@theweipeng theweipeng self-assigned this Aug 1, 2023
@theweipeng theweipeng merged commit aa0c02a into apache:main Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[JavaScript] JavaScript should support polymorphism

1 participant