Yes, I agree.
It gets to web.xml indirectly by inclusion in xml/src/jakartaee_10.xsds
which is included by web-common_6_0
which is included by web-app_6_0
I did it this way to most closely follow the precedent that was established by existing application-defined resource types such as data-source and connection-factory. If this isn’t correct, we can move it.
A unit test in the platform schema project validates that it can be put in web.xml,
https://github.com/eclipse-ee4j/jakartaee-schemas/pull/29/files#diff-ac91fbdf21b900e6655191f25add2f29c087b75353553674e119093f9f097dcf
From:
cu-dev <cu-dev-bounces@xxxxxxxxxxx> on behalf of "Steve Millidge (Payara)" <steve.millidge@xxxxxxxxxxx>
Reply-To: cu developer discussions <cu-dev@xxxxxxxxxxx>
Date: Thursday, March 3, 2022 at 9:25 AM
To: cu developer discussions <cu-dev@xxxxxxxxxxx>
Subject: [EXTERNAL] Re: [cu-dev] Javadoc implies a deployment descriptor
Hi Nathan,
As Concurrency is in the web profile I think we also needed it added to web.xml. What do you think?
Steve
From: cu-dev <cu-dev-bounces@xxxxxxxxxxx>
On Behalf Of Steve Millidge (Payara)
Sent: 03 March 2022 14:43
To: cu developer discussions <cu-dev@xxxxxxxxxxx>
Subject: Re: [cu-dev] Javadoc implies a deployment descriptor
Thanks Nathan I missed that.
Steve,
It’s already in the platform schema,
https://github.com/eclipse-ee4j/jakartaee-schemas/pull/29/files
I added it last year at the same time we added ManagedExecutorDefinition and the other annotations.
Our project TCK doesn’t currently test this because the function is part of the platform schema not the Concurrency spec, and I had been thinking that any testing which gets done for it would need to be in the
platform tests. However, over the past half year or so there seems to be a greater acceptance within the community of moving platform level testing to the individual spec project TCKs (and then deleting the platform tests), so we could potentially do that.
Hi,
The Javadoc for ManagedExecutorDefinition states a deployment descriptor can be used to create a ManagedExecutor see
concurrency-api/ManagedExecutorDefinition.java at master · eclipse-ee4j/concurrency-api · GitHub same for ScheduledExecutorDefinition etc. do we want to remove this from the Javadoc or do we want to try and get this into some platform schema somewhere?
I assume the TCK isn’t testing this?
Thanks
Steve Millidge