DataNodeContainer.getDataChildByName() is a @Nullable getter, which is used in that capacity all over the place. On the other hand a number of places would like to make that an assertion.

Fix this by centering the implementation around dataChildByName(), which is nullable and then spinning Optional and non-null from that.