Bugfix and cosmetic changes to resource subsystem
- Continuous resource was always considered not available
- Support querying for child resource against Continuous
(result is no children)
- Resource equality to compare id and exact type
- Add missing information in Continuous resource toString()
- More concise String representation for ResourceId
- additional logging added during above bug investigation.
Change-Id: I58a95b95b91c246c3c5dbb136a1820f988c6fccd
diff --git a/core/net/src/main/java/org/onosproject/net/newresource/impl/ResourceManager.java b/core/net/src/main/java/org/onosproject/net/newresource/impl/ResourceManager.java
index 96466d5..83ae6e0 100644
--- a/core/net/src/main/java/org/onosproject/net/newresource/impl/ResourceManager.java
+++ b/core/net/src/main/java/org/onosproject/net/newresource/impl/ResourceManager.java
@@ -34,12 +34,14 @@
import org.onosproject.net.newresource.Resource;
import org.onosproject.net.newresource.ResourceStore;
import org.onosproject.net.newresource.ResourceStoreDelegate;
+import org.slf4j.Logger;
import java.util.Collection;
import java.util.List;
import java.util.stream.Collectors;
import static com.google.common.base.Preconditions.checkNotNull;
+import static org.slf4j.LoggerFactory.getLogger;
/**
* An implementation of ResourceService.
@@ -53,18 +55,24 @@
@Reference(cardinality = ReferenceCardinality.MANDATORY_UNARY)
protected ResourceStore store;
+ private final Logger log = getLogger(getClass());
+
private final ResourceStoreDelegate delegate = new InternalStoreDelegate();
@Activate
public void activate() {
store.setDelegate(delegate);
eventDispatcher.addSink(ResourceEvent.class, listenerRegistry);
+
+ log.info("Started");
}
@Deactivate
public void deactivate() {
store.unsetDelegate(delegate);
eventDispatcher.removeSink(ResourceEvent.class);
+
+ log.info("Stopped");
}
@Override