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
Updates Service Registry dispatch interval property name and time unit #1120
Updates Service Registry dispatch interval property name and time unit #1120
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @cartucciam, thanks for the patch. Can you please add an explanation of what you are doing and reasoning why you are doing this to the pull request's description?
Also, changing the dispatch interval's property name can easily cause problems if people miss this when upgrading. This does not necessarily mean that we cannot make that change but we should be pretty clear in the upgrade documentation about this.
Hi @lkiesow, that's a good point and makes me wondering if this change is really needed. Can you point me at the correct upgrade documentation? |
I guess docs/guides/admin/docs/upgrade.md would be the right place to announce a change like this. |
Thanks for helping out with this → merged. |
opencast#1120) * Updates Service Registry dispatch interval property name and time unit To be consistent with heartbeat interval ones Fixes opencast#1117
Changing dispatch interval property name and time unit to be consistent with the other settings (as per #1117).
Closes #1117