okulikov/jboss-as — comment on commit c214ffd55b32ce8a99e04f4809e13176e3e68e98
File: jpa/core/src/main/java/org/jboss/as/jpa/puparser/PersistenceUnitXmlParser.java line 196
By jaikiran:

I think we shouldn't be supporting this property replacement for every attribute/element. Instead we should probably support it for only a select set of element/attributes that's decided on a case by case basis. For example, in this case, I don't think it adds value if we allow system properties for the element in the persistence.xml. Same for some other elements/attributes of the xml.