Skip to content

Conversation

@xuzhg
Copy link
Member

@xuzhg xuzhg commented Dec 17, 2025

Issues

This pull request fixes #3440: Missing annotations for null value nested resource or without value nested resource.

Description

When writing the nested resource, we can see the ODL retrieves the instance annotations from ODataNestedResourceInfo and write them as property annotation if the nested resource value is null.

So, this design makes sense but there's no way for developer to specify the instance annotations for ODataNestedResourceInfo since the 'SetInstanceAnnotation()is internal API. This also blocks to read the instance annotations fornull` value nested resource.

This PR is to expose the instance annotations on the ODataNestedResourceInfo. Then

in reading, upper layer can retrieve the instance annotations.

in writing, upper layer can set the instance annotations.

Briefly describe the changes of this pull request.

Checklist (Uncheck if it is not completed)

  • Test cases added
  • Build and test with one-click build and test script passed

Additional work necessary

If documentation update is needed, please add "Docs Needed" label to the issue and provide details about the required document change in the issue.

Repository notes

Team members can start a CI build by adding a comment with the text /AzurePipelines run to a PR. A bot may respond indicating that there is no pipeline associated with the pull request. This can be ignored if the build is triggered.

Team members should not trigger a build this way for pull requests coming from forked repositories. They should instead trigger the build manually by setting the "branch" to refs/pull/{prId}/merge where {prId} is the ID of the PR.

@xuzhg xuzhg changed the base branch from main to dev-9.x December 17, 2025 23:28
@xuzhg
Copy link
Member Author

xuzhg commented Dec 17, 2025

AzurePipelines run

@xuzhg
Copy link
Member Author

xuzhg commented Dec 17, 2025

/AzurePipelines run

@xuzhg
Copy link
Member Author

xuzhg commented Dec 19, 2025

/AzurePipelines run

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.

Missing annotations for null value nested resource or without value nested resource

2 participants