summary |
shortlog |
log |
commit | commitdiff |
review |
tree
raw |
patch |
inline | side by side (from parent 1:
d477357)
Root Tree Node context did not allowed to enter notification
using binding classes by #streamChild method, since
notifications were cached in different collection.
Added support for entering notifications using such method.
Change-Id: I28779e763a81519d25d71e319776cdc6f179ef33
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
@Override
public <DV extends DataObject> DataContainerCodecContext<DV, ?> streamChild(Class<DV> childClass)
throws IllegalArgumentException {
@Override
public <DV extends DataObject> DataContainerCodecContext<DV, ?> streamChild(Class<DV> childClass)
throws IllegalArgumentException {
+ /* FIXME: This is still not solved for RPCs
+ * TODO: Probably performance wise RPC, Data and Notification loading cache
+ * should be merge for performance resons. Needs microbenchmark to
+ * determine which is faster (keeping them separate or in same cache).
+ */
+ if (Notification.class.isAssignableFrom(childClass)) {
+ return (DataContainerCodecContext<DV, ?>) getNotification((Class<? extends Notification>)childClass);
+ }
return (DataContainerCodecContext<DV, ?>) childrenByClass.getUnchecked(childClass);
}
return (DataContainerCodecContext<DV, ?>) childrenByClass.getUnchecked(childClass);
}