| [jakartabatch-dev] Please help review PR 181 for option 2 - status quo - Batch + CDI integration |
OK, so first, apologies if you've only been watching this Batch ML as a lot of recent discussion on Batch+CDI integration in the last couple weeks had moved to the Platform ML, as we sought Platform input.
In spite of the poll still being open, given the time constraints and the late date, I wanted to go ahead and draft a spec update PR for the "Option 2" status quo path.
This is at:
https://github.com/eclipse-ee4j/batch-api/pull/181/
IMHO, I think it's easier to review by building the new spec document and reading it straight through than looking at the diff/compare.
`mvn install -Dmaven.javadoc.skip=true` # output in spec/target/generated-docs
I left some review notes highlight the changed sections and key questions in the GitHub comment.
----
BTW, if anyone is looking to help at this point, I think adding TCK tests covering the newly-specified items would be a helpful way to contribute.
Though this may be the impl status quo.. it is of course not covered by the TCK (well, maybe there's a few assumptions implicitly tested) and a decent amount of new material.
Thanks,
------------------------------------------------------
Scott Kurz
WebSphere / Open Liberty Batch and Developer Experience
skurz@xxxxxxxxxx
--------------------------------------------------------