Github user mike-tutkowski commented on the pull request:
https://github.com/apache/cloudstack/pull/630#issuecomment-128604281
That is very well true, but I "worry" that a comment that specific would
not be updated should the time come when another adaptor does, in fact, return
null here.
Also, I think it's possible for people to provide these adaptors to their
CloudStack users without donating the code to the ASF. That being the case, we
really don't know if anyone's implementation returns null besides that of
NfsManagedStorageAdaptor.
It's certainly not a big deal to me (this comment) either way.
You make the call. You have a LGTM from me. :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
