Description of problem:

I have a plugin that uses the subclass and parent class to obtain measurement values. I wrote the code in the following way:

    @Override
    public void getValues(MeasurementReport mr, Set<MeasurementScheduleRequest> msrs) {
        Map<String, Double> map = new HashMap<String, Double>();
        try {
            getMeasurements(map);
        } catch (Exception e) {
            log.warn("failed to digest additional content", e);
        }
        for (Iterator<MeasurementScheduleRequest> i = msrs.iterator(); i.hasNext(); ) {
            MeasurementScheduleRequest msr = i.next();
            Double d = digest.get(msr.getName());
            if (d != null) {
                mr.addData(new MeasurementDataNumeric(msr, d));
                i.remove(); // XXX
            }
        }
        super.getValues(mr, msrs);
    }

What this does, unfortunately, is unschedule the measurement request.

The "Set" passed into the method should either be read-only or a copy.

It would probably be better to be read-only, though.



Version-Release number of selected component (if applicable):

RHQ 4.2

How reproducible:

Always


Additional info: