[GitHub] spark pull request: [SPARK-3816][SQL] Add table properties from st...

2014-10-27 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-60573570 @marmbrus Yes, it's related. The `testData` was cached by some other test suite executed before `JoinSuite`, and one of the expected `ShuffledHashJoin` was optimized

[GitHub] spark pull request: [SPARK-3816][SQL] Add table properties from st...

2014-10-27 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-60576607 #2960 is filed to fix this issue. --- 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

[GitHub] spark pull request: [SPARK-3816][SQL] Add table properties from st...

2014-10-27 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-60576734 retest this please This failure is sensitive to test suite execution order, so here I'd like to have another try. --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-3816][SQL] Add table properties from st...

2014-10-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-60638532 [Test build #474 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/474/consoleFull) for PR 2677 at commit

[GitHub] spark pull request: [SPARK-3816][SQL] Add table properties from st...

2014-10-27 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-60706146 Thanks! Merged to master. --- 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-3816][SQL] Add table properties from st...

2014-10-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/2677 --- 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-3816][SQL] Add table properties from st...

2014-10-26 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-60534260 Can you please rebase 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 project does not

[GitHub] spark pull request: [SPARK-3816][SQL] Add table properties from st...

2014-10-26 Thread alexliu68
Github user alexliu68 commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-60535574 It's rebased. --- 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-3816][SQL] Add table properties from st...

2014-10-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-60538628 [Test build #472 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/472/consoleFull) for PR 2677 at commit

[GitHub] spark pull request: [SPARK-3816][SQL] Add table properties from st...

2014-10-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-60538935 [Test build #472 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/472/consoleFull) for PR 2677 at commit

[GitHub] spark pull request: [SPARK-3816][SQL] Add table properties from st...

2014-10-26 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-60539054 Hmm, this failure looks related to @liancheng patch on statistics that I just merged. On Oct 26, 2014 5:18 PM, Apache Spark QA notificati...@github.com wrote:

[GitHub] spark pull request: [SPARK-3816][SQL] Add table properties from st...

2014-10-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-60475438 [QA tests have started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/446/consoleFull) for PR 2677 at commit

[GitHub] spark pull request: [SPARK-3816][SQL] Add table properties from st...

2014-10-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-60477986 **[Tests timed out](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/446/consoleFull)** for PR 2677 at commit

[GitHub] spark pull request: [SPARK-3816][SQL] Add table properties from st...

2014-10-22 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-60060432 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 have this

[GitHub] spark pull request: [SPARK-3816][SQL] Add table properties from st...

2014-10-10 Thread alexliu68
Github user alexliu68 commented on the pull request: https://github.com/apache/spark/pull/2677#issuecomment-58681796 Sure, I am pretty busy on my daily work right now. I will find time to add unit test. I fix this issue when I integrate CassandraStorageHandler. --- If your project