Skip to content

Conversation

@adutra
Copy link
Contributor

@adutra adutra commented Dec 26, 2024

This PR removes the CallContextCatalogFactory class that becomes obsolete after the removal of CallContext.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Protected method, in case some tests need to override it (not the case at the moment).

@adutra adutra force-pushed the remove-call-context-catalog-factory branch 16 times, most recently from 91f3ce3 to 29efd5e Compare January 9, 2025 14:00
@adutra adutra force-pushed the remove-call-context-catalog-factory branch 7 times, most recently from cb5fffc to 8e1167a Compare January 13, 2025 20:49
@adutra adutra force-pushed the remove-call-context-catalog-factory branch from 8e1167a to e2c5b6c Compare January 14, 2025 11:14
@adutra adutra changed the title [WIP] Remove CallContextCatalogFactory Remove CallContextCatalogFactory Jan 14, 2025
@adutra adutra marked this pull request as ready for review January 14, 2025 11:16
@adutra
Copy link
Contributor Author

adutra commented Jan 14, 2025

Ready for review.

@adutra adutra merged commit 8a46a45 into apache:main Jan 14, 2025
5 checks passed
@adutra adutra deleted the remove-call-context-catalog-factory branch January 14, 2025 11:33
eric-maynard added a commit to eric-maynard/polaris that referenced this pull request Feb 3, 2025
eric-maynard added a commit to eric-maynard/polaris that referenced this pull request Feb 4, 2025
eric-maynard added a commit that referenced this pull request Feb 4, 2025
* Revert "Remove CallContextCatalogFactory (#592)"

This reverts commit 8a46a45.

* fix conflicts

* spotless
eric-maynard added a commit to eric-maynard/polaris that referenced this pull request Feb 6, 2025
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.

2 participants