[GitHub] spark pull request: [SPARK-11788][SQL] surround timestamp/date val...

2015-12-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9872 --- 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

[GitHub] spark pull request: [SPARK-11788][SQL] surround timestamp/date val...

2015-12-01 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/9872#issuecomment-161131088 Merging to master and branch 1.6. --- 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

[GitHub] spark pull request: [SPARK-11788][SQL] surround timestamp/date val...

2015-12-01 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/9872#issuecomment-161137050 I also merged it in branch 1.5. --- 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

[GitHub] spark pull request: [SPARK-11788][SQL] surround timestamp/date val...

2015-12-01 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9872#issuecomment-160988167 **[Test build #2135 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2135/consoleFull)** for PR 9872 at commit

[GitHub] spark pull request: [SPARK-11788][SQL] surround timestamp/date val...

2015-12-01 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9872#issuecomment-161022692 **[Test build #2135 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2135/consoleFull)** for PR 9872 at commit

[GitHub] spark pull request: [SPARK-11788][SQL] surround timestamp/date val...

2015-11-27 Thread huaxingao
Github user huaxingao commented on the pull request: https://github.com/apache/spark/pull/9872#issuecomment-160182654 Fixed title and scala style problem. Thanks! --- 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

[GitHub] spark pull request: [SPARK-11788][SQL]:surround timestamp/date val...

2015-11-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9872#issuecomment-160034888 **[Test build #2122 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2122/consoleFull)** for PR 9872 at commit

[GitHub] spark pull request: [SPARK-11788][SQL]:surround timestamp/date val...

2015-11-26 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/9872#issuecomment-160034631 LGTM - Jenkins, test this please. --- 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

[GitHub] spark pull request: [SPARK-11788][SQL]:surround timestamp/date val...

2015-11-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9872#issuecomment-160034762 **[Test build #2122 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2122/consoleFull)** for PR 9872 at commit

[GitHub] spark pull request: [SPARK-11788][SQL]:surround timestamp/date val...

2015-11-26 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/9872#issuecomment-160034681 Can you update the title to "[SPARK-11788][SQL] surround timestamp/date value with quotes in JDBC data source" --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-11788][SQL]:surround timestamp/date val...

2015-11-24 Thread huaxingao
Github user huaxingao commented on the pull request: https://github.com/apache/spark/pull/9872#issuecomment-159375392 @JoshRosen Regression test added. Could you please take a look? Thanks a lot for your help!! --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-11788][SQL]:surround timestamp/date val...

2015-11-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9872#issuecomment-158543531 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

[GitHub] spark pull request: [SPARK-11788][SQL]:surround timestamp/date val...

2015-11-20 Thread huaxingao
GitHub user huaxingao opened a pull request: https://github.com/apache/spark/pull/9872 [SPARK-11788][SQL]:surround timestamp/date value with quotes You can merge this pull request into a Git repository by running: $ git pull https://github.com/huaxingao/spark spark-11788

[GitHub] spark pull request: [SPARK-11788][SQL]:surround timestamp/date val...

2015-11-20 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/9872#issuecomment-158544649 Can you add a regression test for this? --- 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

[GitHub] spark pull request: [SPARK-11788][SQL]:surround timestamp/date val...

2015-11-20 Thread huaxingao
Github user huaxingao commented on the pull request: https://github.com/apache/spark/pull/9872#issuecomment-158545092 Sure. Will add a 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