Skip to content

jakarta.persistence.EntityListener XML declaration#1150

Open
sebersole wants to merge 2 commits into
jakartaee:mainfrom
sebersole:xsd-entity-listener
Open

jakarta.persistence.EntityListener XML declaration#1150
sebersole wants to merge 2 commits into
jakartaee:mainfrom
sebersole:xsd-entity-listener

Conversation

@sebersole

Copy link
Copy Markdown
Contributor

Possible solution to #1148. This option uses in-line repeatable elements following the lead from other repeatable elements in this XSD. E.g.

<persistence ...>
    <persistence-unit ....>
        <class>com.acme.A</class>
        <class>com.acme.B</class>

        ...

        <entity-listener>com.acme.Journaler</entity-listener>
        <entity-listener>com.acme.PermissionChecker</entity-listener>
    </persistence-unit>
</persistence>

@gavinking gavinking left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, after the change I suggest below.

Comment thread spec/src/main/asciidoc/ch08-entity-packaging.adoc Outdated
@gavinking gavinking added the 4.0.0 label Jul 2, 2026
@gavinking gavinking linked an issue Jul 2, 2026 that may be closed by this pull request
Co-authored-by: Gavin King <gavin@hibernate.org>

@marko-bekhta marko-bekhta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

maybe worth having some quick test next to

?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

entity-listener XML element

3 participants