Personally i'd rather have this use `ResourceResolver.findResources(...)` so its easy to use the Resource APIs instead of Nodes. Not a big deal either way; The Sling resource APIs tend to provide cleaner, simpler and safer abstractions.

http://sling.apache.org/apidocs/sling5/org/apache/sling/api/resource/ResourceResolver.html#findResources(java.lang.String, java.lang.String)