[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138810632 Thanks all for your review efforts! --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user liancheng closed the pull request at: https://github.com/apache/spark/pull/8583 --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138464068 [Test build #42123 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42123/consoleFull) for PR 8583 at commit [`cd91a26`](https://github.com/apache/spark/commit/cd91a26c86c72f25221ca947c5486a910194a751). --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138462685 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/8583#discussion_r38897017 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetQuerySuite.scala --- @@ -229,4 +232,322 @@ class ParquetQuerySuite extends QueryTest with ParquetTest with SharedSQLContext } } } + + test("SPARK-10301 requested schema clipping - same schema") { +withTempPath { dir => + val path = dir.getCanonicalPath + val df = sqlContext.range(1).selectExpr("NAMED_STRUCT('a', id, 'b', id + 1) AS s").coalesce(1) + df.write.parquet(path) + + val userDefinedSchema = +new StructType() + .add( +"s", +new StructType() + .add("a", LongType, nullable = true) + .add("b", LongType, nullable = true), +nullable = true) + + checkAnswer( +sqlContext.read.schema(userDefinedSchema).parquet(path), +Row(Row(0L, 1L))) +} + } + + ignore("SPARK-10301 requested schema clipping - schemas with disjoint sets of fields") { --- End diff -- This test case is ignored because of a bug probably coming from parquet-mr side. I'm verifying 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138462606 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138501734 [Test build #42123 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42123/console) for PR 8583 at commit [`cd91a26`](https://github.com/apache/spark/commit/cd91a26c86c72f25221ca947c5486a910194a751). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138502075 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138502078 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42123/ 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138519275 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138519290 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138520479 [Test build #42131 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42131/consoleFull) for PR 8583 at commit [`7eb7e09`](https://github.com/apache/spark/commit/7eb7e092d3b4023444d96030815ab0e005e97c10). --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138679516 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138679519 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42139/ 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138684867 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42140/ 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138684865 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138679346 [Test build #42139 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42139/console) for PR 8583 at commit [`85ce339`](https://github.com/apache/spark/commit/85ce339106c4868c3b5a4658caf1b92b737be788). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138684658 [Test build #42140 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42140/console) for PR 8583 at commit [`8812f64`](https://github.com/apache/spark/commit/8812f64ba521b1339bbdb832e1a566c76c047cc4). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138707094 test cases are good. --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138707168 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138560460 [Test build #42131 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42131/console) for PR 8583 at commit [`7eb7e09`](https://github.com/apache/spark/commit/7eb7e092d3b4023444d96030815ab0e005e97c10). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138560611 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138560612 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42131/ 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138769798 ok. It has been merged. --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138769671 Thanks. I am merging it to 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/8583#discussion_r38954416 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetQuerySuite.scala --- @@ -229,4 +232,322 @@ class ParquetQuerySuite extends QueryTest with ParquetTest with SharedSQLContext } } } + + test("SPARK-10301 requested schema clipping - same schema") { +withTempPath { dir => + val path = dir.getCanonicalPath + val df = sqlContext.range(1).selectExpr("NAMED_STRUCT('a', id, 'b', id + 1) AS s").coalesce(1) + df.write.parquet(path) + + val userDefinedSchema = +new StructType() + .add( +"s", +new StructType() + .add("a", LongType, nullable = true) + .add("b", LongType, nullable = true), +nullable = true) + + checkAnswer( +sqlContext.read.schema(userDefinedSchema).parquet(path), +Row(Row(0L, 1L))) +} + } + + ignore("SPARK-10301 requested schema clipping - schemas with disjoint sets of fields") { --- End diff -- Confirmed that this is a parquet-mr bug. --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138645222 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138645467 [Test build #42139 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42139/consoleFull) for PR 8583 at commit [`85ce339`](https://github.com/apache/spark/commit/85ce339106c4868c3b5a4658caf1b92b737be788). --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138645179 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138646407 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138646441 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138648149 [Test build #42140 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42140/consoleFull) for PR 8583 at commit [`8812f64`](https://github.com/apache/spark/commit/8812f64ba521b1339bbdb832e1a566c76c047cc4). --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-138184856 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-137805318 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-137805298 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-137804787 @davies @yhuai I addressed your comments in #8509 here. Will open a separate PR later to fix them for 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user davies commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-137812894 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-137842958 [Test build #42007 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42007/console) for PR 8583 at commit [`cab8701`](https://github.com/apache/spark/commit/cab870114e81996fdf26c710b287081038b7818d). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-137843069 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-137843071 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42007/ 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 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-137804050 @yhuai Done. --- 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
[GitHub] spark pull request: [SPARK-10301] [SPARK-10428] [SQL] [BRANCH-1.5]...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8583#issuecomment-137805892 [Test build #42007 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42007/consoleFull) for PR 8583 at commit [`cab8701`](https://github.com/apache/spark/commit/cab870114e81996fdf26c710b287081038b7818d). --- 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