Github user mateiz commented on the pull request: https://github.com/apache/spark/pull/1658#issuecomment-52113865 BTW apart from that, a few other comments: - Make sure that the classes / objects you add are `private[spark]` so they don't show up in the API docs and such. - You should also design some test suites for this to put in core/src/test. Take a look at other test suites to see how ScalaTest works and how we create temp files. Basically you want to create some files programmatically in the test, then try to read them with this API. Thanks for the continued work on this BTW, it's looking great.
--- 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. --- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org