[jboss-as7-dev] System property replacement in deployment descriptors

Oleg Kulikov oleg.kulikoff at gmail.com
Wed Feb 15 12:04:39 EST 2012


Thanks, I understand the issue now.

--Oleg

15 февраля 2012 г. 19:35 пользователь Brian Stansberry <
brian.stansberry at redhat.com> написал:

> Comments in-line:
>
> On 2/15/12 10:03 AM, Jaikiran Pai wrote:
> > Oleg, sorry about the late response.
> >
> > Currently in AS7 we have xml parsing happening within the AS7 code base
> > for the following xmls:
> >
> > 1) The standalone*.xml/domain*.xml
>
> These files (and host.xml) should be excluded from this effort. They
> have significantly different requirements, primarily related to the need
> to maintain and write back the unresolved value.
>
> > 2) The jboss-deployment-structure.xml
> > 3) The jboss-ejb-client.xml
> > 4) The jboss-pojo xml
> > 5) The jboss-service.xml
> >
> > and probably a few others. Then we have parsers in other projects
> > outside of AS7 codebase which deal with (for example):
> >
> > 1) The spec specified EE descriptors like ejb-jar.xml, web.xml,
> > application.xml
> > 2) JBoss specific (EE) deployment descriptors for the deployments like
> > jboss-web.xml, jboss-app.xml, jboss-ejb3.xml
> >
> > These have their own set of parsers.
> >
> > So obviously trying to _share_ the same system property replacement
> > logic utility class, between these projects isn't going to work out. And
> > since it's just going to be one since class which is going to parse and
> > replace the system property, I think we should just create it in the AS7
> > code base and let the parsers in the AS7 code base use that (whichever
> > parser wants it). The other projects (like jboss-metadata) can use their
> > own (actually we just added one sometime back to support system property
> > replacement for "distinct-name" element in the JBoss specific EE
> > descriptors).
> >
> > By the way, the DMR project has a class which handles this property
> > replacement (in that project). You might want to borrow that relevant
> > code
> >
> https://github.com/jbossas/jboss-dmr/blob/master/src/main/java/org/jboss/dmr/ExpressionValue.java#L110
> .
> >
>
> In a commment on the JIRA I pointed out the old jboss-common-core
> parsing method. But Jaikiran is right to highlight the DMR method as a
> better choice. Scott Stark added logic to it for resolving against the
> VM environment variables (System.getenv()) and not just the system
> properties.
>
> > -Jaikiran
> >
> > On Friday 10 February 2012 02:31 PM, Oleg Kulikov wrote:
> >> Hi Jaikiran,
> >>
> >> Can you explain more details about your vision of the property
> >> replacement task. In general it is interested how deep it should be
> >> shared between different substems where xml descriptors are used.
> >> Should it be a common parsing utility with description which properies
> >> allow expressions or it may be just a simple utility method shared
> >> between parsing methods?
> >>
> >> -- Oleg.
> >>
> >>
> >> _______________________________________________
> >> jboss-as7-dev mailing list
> >> jboss-as7-dev at lists.jboss.org
> >> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
> >
> > _______________________________________________
> > jboss-as7-dev mailing list
> > jboss-as7-dev at lists.jboss.org
> > https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>
>
> --
> Brian Stansberry
> Principal Software Engineer
> JBoss by Red Hat
> _______________________________________________
> jboss-as7-dev mailing list
> jboss-as7-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/jboss-as7-dev/attachments/20120215/3c39f476/attachment.html 


More information about the jboss-as7-dev mailing list