Re: [jakartabatch-dev] Check for the presence of security manager in Bat

[Romain Manni-Bucau <rmannibucau@xxxxxxxxx>]

Le lun. 20 juil. 2020 à 13:20, Scott Kurz <skurz@xxxxxxxxxx> a écrit :

I see this as a fair change to make for EE 9, (though the overall goal is just to move from javax->jakarta).. I see this as a "bug".

Romain I wasn't sure want you meant by "ambiguous" provider, but don't think we've discussed.

Assume I have 2 or 3 providers (thanks transitive deps ;)), which one should be used?

Default in EE is to fail but here it runs and if the impl is not configured, the joboperator can not behave as expected (wrong db or so).

 

Anyway I merged the PR.

Thanks Cheng,
------------------------------------------------------
Scott Kurz
WebSphere Batch and Developer Experience
skurz@xxxxxxxxxx
--------------------------------------------------------

Romain Manni-Bucau ---07/19/2020 12:38:08 AM---+1, seems overdue Side (unrelated but saw checking the pr) note: generally EE provider

From: Romain Manni-Bucau <rmannibucau@xxxxxxxxx>
To: jakartabatch developer discussions <jakartabatch-dev@xxxxxxxxxxx>
Date: 07/19/2020 12:38 AM
Subject: [EXTERNAL] Re: [jakartabatch-dev] Check for the presence of security manager in BatchRuntime
Sent by: jakartabatch-dev-bounces@xxxxxxxxxxx

---

+1, seems overdue

Side (unrelated but saw checking the pr) note: generally EE provider loaders fail when a provider is ambiguous, maybe we should open an issue about it if not already discussed?

Le dim. 19 juil. 2020 à 05:19, Cheng Fang <cfang@xxxxxxxxxx> a écrit :

Hi Jakarta Batch Team,

I recently created the following issue:

Issue #38
Check for the presence of security manager in BatchRuntime

and related PR 40 to batch api repo.

The idea is to bypass the doPriv block in case of no security manager, and hopefully get performance improvement. Given that getJobOperator() can be a frequently-called methods, I think it's worthwhile to do this check.

What do you think?

Cheng
_______________________________________________
jakartabatch-dev mailing list
jakartabatch-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jakartabatch-dev_______________________________________________
jakartabatch-dev mailing list
jakartabatch-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jakartabatch-dev

_______________________________________________
jakartabatch-dev mailing list
jakartabatch-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jakartabatch-dev