case in point, if the caller is responsible for closing here then I think at least the method should be called "createFromDirectory". The API doc should also remind the caller of their responsibility in closing it. Perhaps an empty open method would be good here though.