Moti Asayag has posted comments on this change. Change subject: core: Add JPA infrastructure to oVirt ......................................................................
Patch Set 15: (1 comment) http://gerrit.ovirt.org/#/c/33832/15/backend/manager/modules/bll/src/main/resources/META-INF/persistence.xml File backend/manager/modules/bll/src/main/resources/META-INF/persistence.xml: Line 1: <persistence xmlns="http://java.sun.com/xml/ns/persistence"; Line 2: xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; Line 3: xsi:schemaLocation="http://java.sun.com/xml/ns/persistence http://java.sun.com/xml/ns/persistence/persistence_2_0.xsd"; Line 4: version="2.0"> shouldn't this file be part of the dal project / module ? Line 5: <persistence-unit name="ovirt"> Line 6: <provider>org.hibernate.ejb.HibernatePersistence</provider> Line 7: <non-jta-data-source>java:/ENGINEDataSource</non-jta-data-source> Line 8: <properties> -- To view, visit http://gerrit.ovirt.org/33832 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2440162230449d0c335a2c2d418fb9ff7e854793 Gerrit-PatchSet: 15 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Liran Zelkha <[email protected]> Gerrit-Reviewer: Moti Asayag <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
