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

Reply via email to