Weidong Xu

@luisCavalcantiDev PS: if you are doing advanced work on MSGraph, you might want to use their dedicated SDK here https://github.com/microsoftgraph/msgraph-sdk-java

@XiaofeiCao If those fields are exposed by AADUser model, we probably should set them in `select` on our `list` API. But if these fields are not exposed, user is recommended...

@XiaofeiCao I mean our handwritten model, not the Inner one (we assume user will not use the Inner, if they use, they are on their own).

> @XiaofeiCao > > If those fields are exposed by AADUser model, we probably should set them in `select` on our `list` API. > > But if these fields are...

@luisCavalcantiDev Some context, the `innerModel` is considered an implementation detail, and serves as an escape-hatch -- when user cannot find the property from other getters, they can try query there,...

Pending to be resolved before GA 2. method signature "uploadTestFile" (should we use `Path file` instead of `String fileName, BinaryDate file`) 3. client/subclient structure 4. enable jacoco and confirm test...

None of the SDK is released, hence changing on "OperationId" should be fine (as long as SDK arch board is fine with their SDK). However the removal of "top" parameter...

> Breaking Change Investigation: > > 1. document: [Set SchemaContract.Document as required. #20110](https://github.com/Azure/azure-rest-api-specs/pull/20110) > 2. ApiVersionParameter, ResourceGroupParameter, SubscriptionIdParameter: [[2022-04-01-preview] Add New Api-version for Microsoft.ApiManagement #20399](https://github.com/Azure/azure-rest-api-specs/pull/20399) > 3. allOf : Issue...

please fix ModelValidation

ping me when you ready to merge