[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-09-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/3558 --- 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 enab

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-08-01 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-126942028 This was fixed by #5806, so I think we can close this issue / PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97882197 /cc @marmbrus --- 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 enab

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97860008 [Test build #31422 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31422/consoleFull) for PR 5806 at commit [`6db4bf5`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97860087 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/314

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97860080 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97855667 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97855596 [Test build #31421 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31421/consoleFull) for PR 5806 at commit [`7fc4a98`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97855674 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/314

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97818339 [Test build #31422 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31422/consoleFull) for PR 5806 at commit [`6db4bf5`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97817733 Merged build triggered. --- 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 ha

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97817791 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97816395 [Test build #31421 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31421/consoleFull) for PR 5806 at commit [`7fc4a98`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97815883 Merged build triggered. --- 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 ha

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97815915 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97732087 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/314

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97732074 [Test build #31410 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31410/consoleFull) for PR 5806 at commit [`d5a9933`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97732086 Merged build finished. Test FAILed. --- 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

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97712629 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97712669 [Test build #31410 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31410/consoleFull) for PR 5806 at commit [`d5a9933`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5806#issuecomment-97712608 Merged build triggered. --- 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 ha

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-30 Thread scwf
GitHub user scwf opened a pull request: https://github.com/apache/spark/pull/5806 [SPARK-4699][SQL] make caseSensitive configurable in Analyzer.scala You can merge this pull request into a Git repository by running: $ git pull https://github.com/scwf/spark case Alternatively

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-04-02 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-89087424 ping. Thanks for working on this! However, to keep the PR queue small, I propose we close this issue until you have time to update it. --- If your project is

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-03-25 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/3558#discussion_r27184692 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystConf.scala --- @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software F

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-03-25 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/3558#discussion_r27184655 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystConf.scala --- @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software F

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-03-25 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/3558#discussion_r27184671 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -39,11 +40,17 @@ object SimpleAnalyzer extends Analyzer(E

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-02-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-75378528 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-75378526 [Test build #27816 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27816/consoleFull) for PR 3558 at commit [`664d1e9`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-75378104 [Test build #27815 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27815/consoleFull) for PR 3558 at commit [`12eca9a`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-02-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-75378106 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-75376882 [Test build #27816 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27816/consoleFull) for PR 3558 at commit [`664d1e9`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-02-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-75376674 [Test build #27815 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27815/consoleFull) for PR 3558 at commit [`12eca9a`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-72705867 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-72705843 [Test build #26660 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26660/consoleFull) for PR 3558 at commit [`39e369c`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-72698496 [Test build #26660 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26660/consoleFull) for PR 3558 at commit [`39e369c`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-21 Thread jackylk
Github user jackylk commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-70803953 Please run the test again. I tested all sql and hive test case locally, and no error is reported. I am not sure why online test fails --- If your project is set up for i

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-70473988 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-70473982 [Test build #25755 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25755/consoleFull) for PR 3558 at commit [`dee56e9`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-70469261 [Test build #25755 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25755/consoleFull) for PR 3558 at commit [`dee56e9`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-70459454 [Test build #25751 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25751/consoleFull) for PR 3558 at commit [`05b09a3`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-70459458 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-70456370 [Test build #25751 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25751/consoleFull) for PR 3558 at commit [`05b09a3`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-18 Thread jackylk
Github user jackylk commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-70456295 I have updated the code based on SPARK-3965 (SPARK-5168) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-10 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-69476078 I think this should probably go in after #3965. We can have some parent trait in catalyst that the SQL version can extend. That way we can just pass the configuration

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68603019 [Test build #25015 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25015/consoleFull) for PR 3558 at commit [`73c16b1`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68603021 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68601740 [Test build #25015 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25015/consoleFull) for PR 3558 at commit [`73c16b1`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68598122 [Test build #25014 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25014/consoleFull) for PR 3558 at commit [`9bf4cc7`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68598124 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68596910 [Test build #25014 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25014/consoleFull) for PR 3558 at commit [`9bf4cc7`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68596320 [Test build #25013 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25013/consoleFull) for PR 3558 at commit [`005c56d`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68596321 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68596085 [Test build #25012 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25012/consoleFull) for PR 3558 at commit [`6332e0f`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68596087 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68596075 [Test build #25013 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25013/consoleFull) for PR 3558 at commit [`005c56d`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68595828 [Test build #25012 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25012/consoleFull) for PR 3558 at commit [`6332e0f`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68595566 [Test build #25011 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25011/consoleFull) for PR 3558 at commit [`8753610`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68595569 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2015-01-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68595324 [Test build #25011 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25011/consoleFull) for PR 3558 at commit [`8753610`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-30 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-68394247 ping --- 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 a

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67745356 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67745351 [Test build #24673 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24673/consoleFull) for PR 3558 at commit [`578d167`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67743789 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67743788 [Test build #24675 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24675/consoleFull) for PR 3558 at commit [`fcbf0d9`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67743445 [Test build #24675 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24675/consoleFull) for PR 3558 at commit [`fcbf0d9`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread jackylk
Github user jackylk commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67743335 modified according to @marmbrus, a CatalystConf trait is added and used in Analyzer and Catalog --- If your project is set up for it, you can reply to this email and hav

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67742932 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67742930 [Test build #24674 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24674/consoleFull) for PR 3558 at commit [`e7bca31`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67742900 [Test build #24674 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24674/consoleFull) for PR 3558 at commit [`e7bca31`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67742081 [Test build #24673 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24673/consoleFull) for PR 3558 at commit [`578d167`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67741939 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67741937 [Test build #24672 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24672/consoleFull) for PR 3558 at commit [`210915f`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67741917 [Test build #24672 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24672/consoleFull) for PR 3558 at commit [`210915f`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67741434 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67741433 [Test build #24671 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24671/consoleFull) for PR 3558 at commit [`91b1b96`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-20 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67741385 [Test build #24671 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24671/consoleFull) for PR 3558 at commit [`91b1b96`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-17 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-67412045 I like the idea of having a configuration interface in catalyst that SQLConf can inherit from. We can then pass that into the analyzer. I do want the HiveContex

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-09 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-66291005 That's a good point, Can we make another interface that SqlConf inherit from? Or can we move the SqlConf into the module `Catalyst`? Any other idea? --- If your p

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-06 Thread jackylk
Github user jackylk commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-65892057 If we go for second way, it will create cyclic dependency between spark-catalyst and spark-sql sub-projects, because SQLConf and SQLContext is in spark-sql while Analyzer

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-02 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-65346490 And it will be nice if you can also add test suite for this. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-02 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-65345962 Great, you're right, please make the change. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-02 Thread jackylk
Github user jackylk commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-65340071 @chenghao-intel agreed. Besides, I think the caseSensitive passing to SimpleCatalog also needs to be changed here: https://github.com/apache/spark/blob/master/s

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-02 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-65334998 The change LGTM. --- 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

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-02 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-65334620 I think we probably have to always keep the HiveContext analyzer case INSENSITIVE, See https://cwiki.apache.org/confluence/display/Hive/LanguageManual+DDL, Hive wi

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-65292399 [Test build #24051 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24051/consoleFull) for PR 3558 at commit [`578d167`](https://gith

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-65292413 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-02 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-65281697 [Test build #24051 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24051/consoleFull) for PR 3558 at commit [`578d167`](https://githu

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-02 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-65281098 ok to test --- 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 ena

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-02 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-65281087 Thanks for working on this! What about HiveContext? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If y

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-02 Thread jackylk
GitHub user jackylk opened a pull request: https://github.com/apache/spark/pull/3558 [SPARK-4699][SQL] make caseSensitive configurable in Analyzer.scala You can merge this pull request into a Git repository by running: $ git pull https://github.com/jackylk/spark case Alternat

[GitHub] spark pull request: [SPARK-4699][SQL] make caseSensitive configura...

2014-12-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3558#issuecomment-65270774 Can one of the admins verify this patch? --- 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 pro