Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jakarta Batch 2.0 #238

Merged
merged 9 commits into from Aug 30, 2020
Merged

Jakarta Batch 2.0 #238

merged 9 commits into from Aug 30, 2020

Conversation

scottkurz
Copy link
Contributor

@scottkurz scottkurz commented Jul 26, 2020

NOTE: At this point it appears that we still need the CDI (Wave 3) and JTA (Wave 4, like Batch) dependencies to be staged

Specification PR template

When creating a specification project release review, create PRs with the content defined as follows.

Include the following in the PR:

===================================================================================

Signed-off-by: Scott Kurz <skurz@us.ibm.com>
Signed-off-by: Scott Kurz <skurz@us.ibm.com>
@netlify
Copy link

netlify bot commented Jul 26, 2020

Deploy preview for jakartaee-specifications ready!

Built with commit ebbe3c0

https://deploy-preview-238--jakartaee-specifications.netlify.app

Signed-off-by: Scott Kurz <skurz@us.ibm.com>
@scottkurz scottkurz changed the title Jakarta Batch 2.0 (Specification & Javadoc) Jakarta Batch 2.0 Jul 26, 2020
@kwsutter kwsutter added draft Work in Progress wave:4 Used for release tracking purposes labels Jul 26, 2020
@kwsutter kwsutter added this to the Jakarta EE 9 milestone Jul 26, 2020
@kwsutter kwsutter requested a review from kazumura July 26, 2020 19:31
@kazumura
Copy link
Contributor

kazumura commented Jul 26, 2020

Spec Review Checklist

  1. Spec PR
  1. _index.md
  1. javadocs
  • Footer contains Eclipse copyright and link to license
  • ESFL license is included, usually as doc-files/speclicense.html
  • no META-INF directory in PR
  • javadocs-jar artifact matches apidocs (optional for this release)
  1. Spec PDF
  • Correct spec title
  • Version number of the form x.y, not x.y.z
  • Correct Eclipse copyright line
  • No DRAFT or SNAPSHOT
  • Correct Logo
  1. Spec HTML
  • Same as PDF
  1. TCK zip file
  • README file (optional for this release)
  • EFTL license file, preferably named LICENSE.md
  • User's Guide (or equivalent documentation)
  • How to test the Compatible Implementation(s) listed in _index.md above with the TCK (may be in UG)
  1. TCK User's Guide (or equivalent documentation)
  • Software requirements listed
  • Installation and configuration described
  • How to run tests
  • Where to file challenges
  1. Compatibility certification request
  • Request follows template
  • SHA-256 fingerprint matches staged TCK zip file
  • Request issue has certification label.
  1. TCK results summary
  • Page is hosted by Compatible Implementation project
  • Includes all information from certification request
  • Summary includes number of tests passed, failed, errors
  • SHA-256 fingerprint matches staged TCK zip file on cert request
  1. Update Jakarta EE API jar
  • Update the Jakarta EE API jar by submitting a PR to the jakartaee-api project that updates the version number of your API jar file.

Signed-off-by: Scott Kurz <skurz@us.ibm.com>
@scottkurz scottkurz marked this pull request as ready for review July 31, 2020 01:47
@scottkurz
Copy link
Contributor Author

@kazumura, I think this is ready to continue reviewing (thank you for the review so far).
I do have a PR on the jakartaee-api project here: jakartaee/jakartaee-api#71

Note on my compatible impl for certification request

jakartaee/batch#110

I know this isn't the first situation like this, but there's a bit of a circular dependency here: the final release of the 'jbatch' impl we use to certify has a dependency on the spec/API, but the certifying impl is needed to finalize the spec/API.

I thought the most useful thing to do at this point was to write up the certification issue as if this were the final 2.0.0 release of the 'jbatch' implementation, so you'll see the 2.0.0 version mentioned in the certification issue.

I would expect to come back and update the certification issue when the specification is complete, at which point I can perform the final release of the 2.0.0 'jbatch' implementation.

However, another alternative would be to simply use a "Release Candidate" type of release of 'jbatch' as the certifying implementation. I could rework the jakartaee/batch#110 issue if that is preferred. One downside of that is that I would have to use a pre-final version of the 'jbatch' dependency on JTA.

@kwsutter, please let me know if you have any thoughts here, and if the specification process has any more detailed guidance I should be following for a case like this. If I'm overthinking this that's fine too. I think there's enough transparency captured in the combination of the batch-api issue and the impl issue.

Thank you!

Signed-off-by: Scott Kurz <skurz@us.ibm.com>
@kwsutter
Copy link
Contributor

kwsutter commented Aug 5, 2020

@kwsutter, please let me know if you have any thoughts here, and if the specification process has any more detailed guidance I should be following for a case like this. If I'm overthinking this that's fine too. I think there's enough transparency captured in the combination of the batch-api issue and the impl issue.

You could go the RC route, if you wish. Just so that it sticks around in a public location (ie. maven or github or downloads). Other components are using "Alpha" or "Milestone" drivers in the same light. Whatever works for your testing and certification. Thanks.

Signed-off-by: Scott Kurz <skurz@us.ibm.com>
Signed-off-by: Scott Kurz <skurz@us.ibm.com>
Copy link
Contributor

@kazumura kazumura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks, @scottkurz

@kazumura kazumura added ballot Delivered to the Specification Committee for ballot final Ready for Vote and removed draft Work in Progress wave:4 Used for release tracking purposes labels Aug 11, 2020
@kwsutter kwsutter added the wave:4 Used for release tracking purposes label Aug 11, 2020
@kwsutter
Copy link
Contributor

@scottkurz The pdf version of the Spec doesn't seem to be readable. The html is rendered just fine, but I can't read the pdf with any type of viewer.

@kazumura
Copy link
Contributor

@scottkurz The pdf version of the Spec doesn't seem to be readable. The html is rendered just fine, but I can't read the pdf with any type of viewer.

The pdf file committed at Jul 26 was OK, but the pdf file committed at Jul 31 seem to be html.

Signed-off-by: Scott Kurz <skurz@us.ibm.com>
Signed-off-by: Scott Kurz <skurz@us.ibm.com>
@scottkurz
Copy link
Contributor Author

@kazumura , @kwsutter

I renamed to jakarta-batch-spec-2.0.pdf/html and fixed the broken PDF. Within the spec document I fixed a bunch of broken links to sections within the doc.

I re-staged the API jar as well since it's easier for me to keep the spec+API both updated together in the Maven staging repo. So I copied the newly-generated Javadoc, which should be equivalent to the old, into this PR so it will match what gets released to Maven. Then I re-ran the TCK too against the new API jar.

Thanks for your review.

@kwsutter kwsutter requested a review from kazumura August 12, 2020 23:52
@kwsutter
Copy link
Contributor

Thanks, @scottkurz. Since Scott updated several items in this PR, I think @kazumura needs to do another final re-review before it's ready for ballot. I just requested another review from Kenji. I won't remove any of the labels since I think this will be done in short order before sending out the ballot. Thanks!

@kwsutter kwsutter self-requested a review August 12, 2020 23:55
Copy link
Contributor

@kwsutter kwsutter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both the PDF and HTML are readable now. Thanks, @scottkurz!

Copy link
Contributor

@kazumura kazumura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

jeanouii
jeanouii previously approved these changes Aug 25, 2020
@jeanouii jeanouii dismissed their stale review August 26, 2020 16:13

Question on the zip name

Copy link
Contributor

@jeanouii jeanouii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comment on the zip file name.
Not sure it's a blocker

* [Jakarta Batch 2.0 Javadoc](./apidocs)
* [Jakarta Batch 2.0 TCK](https://download.eclipse.org/jakartabatch/tck/eftl/jakarta.batch.official.tck-2.0.0-M2.zip) (sig - N/A, [sha](https://download.eclipse.org/jakartabatch/tck/eftl/jakarta.batch.official.tck-2.0.0-M2.zip.sha256), pub - N/A):
* [Jakarta Batch 2.0 TCK](https://download.eclipse.org/jakartaee/batch/2.0/jakarta.batch.official.tck-2.0.0.zip) ([sig](https://download.eclipse.org/jakartaee/batch/2.0/jakarta.batch.official.tck-2.0.0.zip.sig), [sha](https://download.eclipse.org/jakartaee/batch/2.0/jakarta.batch.official.tck-2.0.0.zip.sha256), [pub](https://raw.githubusercontent.com/jakartaee/specification-committee/master/jakartaee-spec-committee.pub))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is there an "official" in the zip name?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the Spec. committee meeting at Aug 12, we discussed about the form of zip file name,
and we agreed that the folder path is required, the file name pattern is preferred.

@kazumura
Copy link
Contributor

kazumura commented Aug 30, 2020

  • The specification committee mentor adds this final checklist to the main PR.
  • The specification committee member adds the approved label to the PRs, and sends out the Ballot Summary per this template to the public Jakarta EE Specification Committee email list
  • The specification committee mentor merges the specification (and apidocs) PRs, ensuring the "date:" field in the _index.md file has an appropriate value to allow publishing.
  • The specification committee mentor calculates the staged EFTL TCK signature and promotes it to the committee download area
    using the https://ci.eclipse.org/jakartaee-spec-committee/job/promote-release/ job.
  • The specification project member who created the api staging release promotes the specification api jars to maven central. An example release job script can be found here https://wiki.eclipse.org/MavenReleaseScript.
  • The specification committee mentor updates the specification page with the ballot results.
    This list goes on the committed spec index page.
  • The specification project team should go through the merged spec website page to verify all the links are valid.
  • The specification project team should approve the compatibility request.
  • The compatible implementation project/vendor MUST send an email to tck@eclipse.org for approval of the compatible implementation for trademark usage.
  • The specification project team should merge any final release branch as appropriate for the branch management for the project.

@kazumura kazumura added the approved The ballot was approved by the Specification Committee label Aug 30, 2020
@kazumura kazumura merged commit 0c9bbc3 into jakartaee:master Aug 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved The ballot was approved by the Specification Committee ballot Delivered to the Specification Committee for ballot final Ready for Vote wave:4 Used for release tracking purposes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants