Skip to content

API review bundle #28504

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jul 26, 2022
Merged

API review bundle #28504

merged 1 commit into from
Jul 26, 2022

Conversation

ajcvickers
Copy link
Contributor

Part of #27588.

  • Remove params, Obsolete, and add a new ctor to IndexAttribute
  • Use entity instead of instance in IIdentityResolutionInterceptor and IMaterializationInterceptor
  • Use parameter objects in IIdentityResolutionInterceptor and IInstantiationBindingInterceptor
  • Remove entityInstanceName parameter in IInstantiationBindingInterceptor.ModifyBinding
  • Rename SkippingIdentityResolutionInterceptor to IgnoringIdentityResolutionInterceptor
  • Remove IQueryExpressionInterceptor.CompilingQuery and CompiledQuery
  • Rename IQueryExpressionInterceptor.ProcessingQuery to QueryCompilationStarting

Part of #27588.

- Remove params, Obsolete, and add a new ctor to IndexAttribute
- Use entity instead of instance in IIdentityResolutionInterceptor and IMaterializationInterceptor
- Use parameter objects in IIdentityResolutionInterceptor and IInstantiationBindingInterceptor
- Remove entityInstanceName parameter in IInstantiationBindingInterceptor.ModifyBinding
- Rename SkippingIdentityResolutionInterceptor to IgnoringIdentityResolutionInterceptor
- Remove IQueryExpressionInterceptor.CompilingQuery and CompiledQuery
- Rename IQueryExpressionInterceptor.ProcessingQuery to QueryCompilationStarting
@ajcvickers ajcvickers requested a review from a team July 25, 2022 09:15
@ajcvickers ajcvickers merged commit 024fd3b into main Jul 26, 2022
@ajcvickers ajcvickers deleted the AyeAyeCaptain0724 branch July 26, 2022 09:54
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants