#3998 [HHQ-3779] Ensure enabled measurements do not have an interv

[Hyperic Team]

http://jira.hyperic.com/browse/HHQ-3779


No steps yet on how to reproduce, but an interval of 0 for an enabled measurement causes those resources to not be viewable in the UI and causes the backfiller to not work properly.

I'd like for us to review the areas of code that touch the interval column. How can it be set to zero? Since all the other areas of the code (such as TimingVoodoo) require it to be non-zero, perhaps something like the following should be added to Measurement

setInterval(long interval) {


  if (isEnabled()) {


    assert(interval != 0) // Or throw IllegalArgumentException("Interval of 0 not allowed for enable measurements")


  }


}

In addition, add validation to the pages where intervals are set. Currently it allows a value of 0, but appears to do nothing but a reschedule of metrics at the current intervals.