Github user zhzhan commented on the pull request: https://github.com/apache/spark/pull/2576#issuecomment-58433545 I have a high level comments. There is another gira spark-2883 opened on 06/Aug/14, which I am actively working on. This gira should be duplicated to that one. To avoid redundant work, I don't think it is a good way open a gira right before sending out PR if the PR is not trial. We have to collaborate in the future to avoid such things. In addition, Orc and sql should be decoupled. I do think this PR is premature, as pointed out in the review comment, it actually introduce the dependency between sql core and hive. By the way, if you are interested, we can collaborate.
--- 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