remove this code before release - from InventoryManager: // TODO REMOVE THIS IF STATEMENT - IT IS JUST FOR TESTING if (!resource.getChildResources().getClass().getName().contains("Collections$SetFromMap")) { new Exception("BAD CHILD SET:" + resource.getChildResources().getClass().getName() + ":" + resource.getId() + ":" + resource.getName()).printStackTrace(); }
Setting priority to urgent and adding to sprint tracker, this obviously doesn't need to be done now, but does need addressing prior to the RHQ4.4 release.
this was for helping to debug bug #801432
rather than remove this, we should wrap it in LOG.isDebugEnabled() or just log it always.
rather than remove this, I wrapped it in a isDebugEnabled and logged it at the debug level: if (log.isDebugEnabled()) { if (!resource.getChildResources().getClass().getName().contains("Collections$SetFromMap")) { Exception e = new Exception( "Unexpected child set - if you see this, please notify support or log it in bugzilla" + resource.getChildResources().getClass().getName() + ":" + resource.getId() + ":" + resource.getName()); log.debug("[BZ 801432]", e); } }
git commit to master: ac24e5c nothing for QA to test - this is just a code change
Bulk closing of items that are on_qa and in old RHQ releases, which are out for a long time and where the issue has not been re-opened since.