Skip to content

Conversation

@snazy
Copy link
Member

@snazy snazy commented Nov 7, 2025

Introduces handling for realms including realm-state management/transition.

The RealmStore implementation for NoSQL depends on CDI components, coming in a follo-up PR.

Introduces handling for realms including realm-state management/transition.

The `RealmStore` implementation for NoSQL depends on CDI components, coming in a follo-up PR.
<beans xmlns="https://jakarta.ee/xml/ns/jakartaee"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="https://jakarta.ee/xml/ns/jakartaee https://jakarta.ee/xml/ns/jakartaee/beans_4_0.xsd">
<!-- File required by Weld (used for testing), not by Quarkus -->
Copy link
Contributor

Choose a reason for hiding this comment

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

sorry, I keep forgetting... why are these test resources in the main src tree? 😅

Copy link
Member Author

Choose a reason for hiding this comment

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

B/c a beans.xml needs to be in every classpath element (jar/folder) that shall be scanned for CDI stuff.
Quarkus doesn't require this, but Weld (CDI reference impl, used for faster testing purposes) does.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, so the beans.xml file has to be located in the same artifact as the beans it references... I see.

@github-project-automation github-project-automation bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board Nov 10, 2025
@dimas-b
Copy link
Contributor

dimas-b commented Nov 10, 2025

This is a well-isolated change. It does not affect any other Polaris code paths and it has been in review since Fri. I'll be merging today unless there are new comments.

@dimas-b dimas-b merged commit 7688a55 into apache:main Nov 10, 2025
15 checks passed
@github-project-automation github-project-automation bot moved this from Ready to merge to Done in Basic Kanban Board Nov 10, 2025
@snazy snazy deleted the nosql-realms-1 branch November 11, 2025 06:58
MonkeyCanCode pushed a commit to MonkeyCanCode/polaris that referenced this pull request Nov 12, 2025
Introduces handling for realms including realm-state management/transition.

The `RealmStore` implementation for NoSQL depends on CDI components, coming in a follo-up PR.
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