Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Adding support for persistence.xml descriptor version 2.1 #574

Merged
merged 1 commit into from May 12, 2014
Merged

Adding support for persistence.xml descriptor version 2.1 #574

merged 1 commit into from May 12, 2014

Conversation

Restage
Copy link

@Restage Restage commented Apr 15, 2014

I've changed the PersistenceXMLLoader to be able to handle version="2.1" of the persistence.xml descriptor.

The code that reads the persistence.xml into an object and verifies it against the target schema has been rewritten to be more generic. You can now add a new schema by just putting it's xsd schema file into the classpath. The validate method than looks into the persistence.xml file and reads the version attribute to determine which xsd to use.

…persistence xml descriptors

The code that reads the persistence.xml into an object and verifies it against the target schema has been rewritten to be more generic. You can now add a new schema by just putting it's xsd schema file into the classpath. The validate method than looks into the persistence.xml file and reads the version attribute to determine which xsd to use.
@mevivs mevivs added this to the 2.12 release milestone Apr 18, 2014
@vishri vishri added the Discuss label May 7, 2014
mevivs added a commit that referenced this pull request May 12, 2014
Adding support for persistence.xml descriptor version 2.1
@mevivs mevivs merged commit 9f7ea31 into Impetus:trunk May 12, 2014
@mevivs
Copy link
Collaborator

mevivs commented May 12, 2014

Merged. Changes looks good to me.
Thanks for your contribution!

-Vivek

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.

None yet

4 participants