Re: [jakartabatch-dev] BatchCDIInjectionExtension.foundJobOp field

[Romain Manni-Bucau <rmannibucau@xxxxxxxxx>]

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 a static foundJobOp to check if we need to create a built-in JobOperator producer:

https://github.com/WASdev/standards.jsr352.jbatch/blob/95cd0cf495c39426f4522747460b3b3a29e91068/com.ibm.jbatch.container/src/main/java/com/ibm/jbatch/container/cdi/BatchCDIInjectionExtension.java#L45

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