Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [jakartaee-platform-dev] [jakarta.ee-spec] Dependency Analysis with jQA - Cyclic Dependency in all Platform Profiles!

For EE 10, the Jakarta Tansactions project was not updated much beyond the EE9 version except some minor changes that went into a service release so there is no new major version planned for EE 10. That said, I don't understand why the Jakarta Transactions API is required to upgrade it's the CDI dependency (nor why compiling with JDK 8 is a problem, unless it would be a problem if we did need to upgrade CDI dependency).

At the Java level, Jakarta Transactions dependencies are: jakarta.enterprise.context.NormalScope [1], jakarta.enterprise.util.Nonbinding [2] [3] and jakarta.interceptor.InterceptorBinding [3]. These APIs still exist [5] [6] [7] but does the Jakarta Transactions inclusion of older API versions cause some problem?

[1] https://github.com/jakartaee/transactions/blob/2.0.1/api/src/main/java/jakarta/transaction/TransactionScoped.java#L65
[2] https://github.com/jakartaee/transactions/blob/2.0.1/api/src/main/java/jakarta/transaction/Transactional.java#L146
[3] https://github.com/jakartaee/transactions/blob/2.0.1/api/src/main/java/jakarta/transaction/Transactional.java#L158
[4] https://github.com/jakartaee/transactions/blob/2.0.1/api/src/main/java/jakarta/transaction/Transactional.java#L59
[5] https://github.com/eclipse-ee4j/cdi/blob/master/api/src/main/java/jakarta/enterprise/context/NormalScope.java
[6] https://github.com/eclipse-ee4j/cdi/blob/master/api/src/main/java/jakarta/enterprise/util/Nonbinding.java
[7] https://github.com/jakartaee/interceptors/blob/master/api/src/main/java/jakarta/interceptor/InterceptorBinding.java


On Mon, 16 May 2022 at 06:39, Matej Novotny <manovotn@xxxxxxxxxx> wrote:
Hmm, I was under the assumption that the tx version in CDI was the new one for EE 10 but apparently there has been no major so far (wasn't that mandatory?).

The dep was in place for years now and it wasn't an issue but I understand that cyclic dep is not nice.

I'll send a PR for removal of that dependency but it might hang in there until Tue since that's the CDI mtg day and it might get some attention there.

Regards
Matej

On Sat, May 14, 2022 at 5:31 PM arjan tijms <arjan.tijms@xxxxxxxxx> wrote:
Hi,

On Sat, May 14, 2022 at 2:07 PM Jan Westerkamp <jan.westerkamp@xxxxxxx> wrote:
Hi Tom,

I agree - Transactions has CDI as valid dependency (right direction, higher level to lower), the core problem is the required dependency of CDI to an already outdated version of Transactions - for documentation reasons only.

Maybe we can ask Matej & Ladislav (cc'ed) to remove this?

Kind regards,
Arjan Tijms



 

Back to the top