adutra opened a new pull request, #2233: URL: https://github.com/apache/polaris/pull/2233
As discussed in the ML here is a PR that merges polaris-service-common into polaris-runtime-service. 99% of the changes are file moves or renamings: - d15c8265a113358696a57c2d873bf97b008e0b33 merges modules - 6df5c2704fca38cac77c15d13d216d4e9fa0d2dd merges packages - 1c8e65b382942c85ad854455342ab73379fa8931 renames classes The 3rd commit merges similar classes like `QuarkusAuthenticationConfiguration` into `AuthenticationConfiguration`, instead of keeping both. One particular interface was problematic: `FeaturesConfiguration` and its subtype `ResolvedFeaturesConfiguration`. The merger resulted in a cyclic reference when resolving CDI beans. The same issue also happened with `BehaviorChangesConfiguration` and `ResolvedBehaviorChangesConfiguration`. To solve both cyclic reference issues, I created the following hierarchy: <img width="1277" height="241" alt="image" src="https://github.com/user-attachments/assets/299ec93c-a764-4b9e-8320-c1ea08f1c419"; /> BTW @eric-maynard what is `BehaviorChangesConfiguration` used for? It seems unused. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
