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

Tweak timer doc #7175

Closed
rtreacy opened this issue Aug 10, 2020 · 0 comments · Fixed by #7181
Closed

Tweak timer doc #7175

rtreacy opened this issue Aug 10, 2020 · 0 comments · Fixed by #7181

Comments

@rtreacy
Copy link
Contributor

rtreacy commented Aug 10, 2020

Although I still think using ejb-jar.xml deployment descriptors to reconfigure automatic (@schedule) timers should be a rare one-off, because of the complexity it re-introduces in Java EE, now that it's out in public documentation, I want to make sure the docs are as accurate and clear as possible, while we're thinking of it.

There's an additional way to besides compiling the app to inject an edited ejb-jar.xml file into the application.

Also when I was assembling the documentation, I inserted the wrong sample file, which had a version problem, which would prevent the dataverse war file from deploying if the ejb-jar.xml file was inserted. I discovered this while re-testing some things after some issues had come up with the merge

@rtreacy rtreacy self-assigned this Aug 10, 2020
mheppler added a commit that referenced this issue Aug 12, 2020
@mheppler mheppler moved this from IQSS Team - In Progress 💻 to QA 🔎✅ in IQSS/dataverse (TO BE RETIRED / DELETED in favor of project 34) Aug 12, 2020
IQSS/dataverse (TO BE RETIRED / DELETED in favor of project 34) automation moved this from QA 🔎✅ to Done 🚀 Aug 12, 2020
@mreekie mreekie added D: SonarQubeCleanup Issues related to cleanup based on Sonar Cube tool and removed D: SonarQubeCleanup Issues related to cleanup based on Sonar Cube tool labels Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants