Github user franz1981 commented on the issue:
https://github.com/apache/activemq-artemis/pull/1596
@clebertsuconic I do not agree with that: the change I did was validated
with tests and the documentation says that when the initial delay isn't
specified is defaulted to period, so the expectation on it is that it will be
defaulted to period in a `getInitialDelay` too.
As I said, for me it's the same but IMO if you want to add a new semantic
over a null initialDelay you need to specify it in some way (ie docs/tests
and/or even better, on the API) in order to avoid subtle bugs and/or to avoid
to change tests that are only checking a get after a set.
---
