Skip to content

Comments

Support multi-level polymorphism#2781

Merged
alzimmermsft merged 5 commits intoAzure:mainfrom
alzimmermsft:MultiLevelPolymorphismSupport
May 24, 2024
Merged

Support multi-level polymorphism#2781
alzimmermsft merged 5 commits intoAzure:mainfrom
alzimmermsft:MultiLevelPolymorphismSupport

Conversation

@alzimmermsft
Copy link
Member

Adds support for multi-level polymorphism, only with stream style serialization.

@haolingdong-msft
Copy link
Member

The changes look good to me.
Would you update the PatchTest to include the multi-level polymorhism case? It can be in this pr or the patch pr https://github.com/Azure/autorest.java/blob/main/typespec-tests/src/test/java/com/cadl/patch/PatchClientTest.java#L123

@weidongxu-microsoft
Copy link
Member

One thing, is that I never tried multi-level discriminator from Swagger. I cannot say whether m4 gives the same code-model.

@alzimmermsft alzimmermsft merged commit 2c72147 into Azure:main May 24, 2024
@alzimmermsft alzimmermsft deleted the MultiLevelPolymorphismSupport branch May 24, 2024 14:39
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.

3 participants