Good catch Cheng,
Thank you for reporting. Opened https://github.com/WASdev/standards.jsr352.jbatch/issues/78 and will do a 2.1.1 release then a PR to pick that up in GF.
From: jakartabatch-dev <jakartabatch-dev-bounces@xxxxxxxxxxx>
On Behalf Of Romain Manni-Bucau
Sent: Thursday, May 12, 2022 3:25 PM
To: jakartabatch developer discussions <jakartabatch-dev@xxxxxxxxxxx>
Subject: [EXTERNAL] Re: [jakartabatch-dev] BatchCDIInjectionExtension.foundJobOp field
Hi, Yes guess so too. Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github | LinkedIn | Book Le jeu. 12 mai 2022 à 20:57, Cheng Fang <cfang@xxxxxxxxxx>
a écrit : I just noticed in JBatch BatchCDIInjectionExtension class, it contains
This Message Is From an External Sender
|
This message came from outside your organization.
|
|
|
I just noticed in JBatch BatchCDIInjectionExtension class, it contains a static foundJobOp to check if we need to create a built-in JobOperator producer:
Should it be an instance variable (instead of a static one)? Image multiple batch apps are deployed to a server concurrently, some with application-defined JobOperator producer, and others don't, and they would interfere with each other.
_______________________________________________
jakartabatch-dev mailing list
jakartabatch-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jakartabatch-dev
|