[GitHub] spark pull request: [SPARK-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-220422809 @davies thank you for your response. Since @rxin categorized this into 2.1, I understand it should be review for 2.1. When 2.0 is out, I will ping you for the review. I would appreciate it if you would recommend other possible reviewers when you have time. --- 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-14098][SQL] Generate Java code that get...
Github user davies commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-220056332 To be honest, Right now is too late to review and merge this into 2.0. I'm busy with other important things on 2.0, does not have enough bandwidth to review it. You could ask other people to look into the design in details. --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-220019065 ping @davies --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-218698658 ping @davies --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217775281 @davies As you suggested, I moved most of the implmentation from ```WholeStageCodegenExec``` to ```InMemoryTableScanExec```. Now, the number of changed lines in ```WholeStageCodegenExec.scala``` is about 30 lines. Since it is hard to pass an instance of ```InMemoryTableScanExec``` to ```InputAdaptor```, I newly introduced ```object InMemoryTableScanExec```. Could you please review this PR? --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217773268 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58117/ 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217773265 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217773178 **[Test build #58117 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58117/consoleFull)** for PR 11956 at commit [`0bc23f8`](https://github.com/apache/spark/commit/0bc23f8934366b7a3eaa899d38fbbd571f5b051a). * 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217766571 **[Test build #58117 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58117/consoleFull)** for PR 11956 at commit [`0bc23f8`](https://github.com/apache/spark/commit/0bc23f8934366b7a3eaa899d38fbbd571f5b051a). --- 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217760415 **[Test build #58115 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58115/consoleFull)** for PR 11956 at commit [`dedca14`](https://github.com/apache/spark/commit/dedca14126696e8b496c12a103368a0d2f1472c1). * This patch **fails to build**. * 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217760423 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58115/ 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 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217760422 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217760139 **[Test build #58115 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58115/consoleFull)** for PR 11956 at commit [`dedca14`](https://github.com/apache/spark/commit/dedca14126696e8b496c12a103368a0d2f1472c1). --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217741844 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217741842 **[Test build #58108 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58108/consoleFull)** for PR 11956 at commit [`e220bf7`](https://github.com/apache/spark/commit/e220bf7c5b3e9620c6cdab4850fd55661ddf8208). * This patch **fails Scala style 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217741846 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58108/ 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217741787 **[Test build #58108 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58108/consoleFull)** for PR 11956 at commit [`e220bf7`](https://github.com/apache/spark/commit/e220bf7c5b3e9620c6cdab4850fd55661ddf8208). --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217241905 I missed something. While I assumed that I would newly declare a singleton object ```InMemoryTableScanExec``` and move 2. and 3. into new methods in the object. As you pointed out, it can be done in a case class. For example, I will move [lines 263-296](https://github.com/apache/spark/pull/11956/files#diff-28cb12941b992ff680c277c651b59aa0R263) and [lines 374-403](https://github.com/apache/spark/pull/11956/files#diff-28cb12941b992ff680c277c651b59aa0R374) into each of two methods in the ```InMemoryTableScanExec``` and then put method calls into the original places in ``WholeStageCodegenExec```. Is this what you want to do? --- 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-14098][SQL] Generate Java code that get...
Github user davies commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217074291 InMemoryTableScanExec is a case class, not singleton, why can we not implement these there? --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217047879 @davies, thank you for your comment. Based on your previous comment, [this commit](https://github.com/apache/spark/pull/11956/commits/d2161d13dab9871ce0e325fcc3934c2f4f5eca13) already moved some stuff of the implementation to ```InMemoryTableScanExec```. Now, ```WholeStageCodegenExec``` does only the following three features. 1. decide whether code for columnar storage should be generated 2. generate ```processRow()``` and ```processBatch``` 3. generate runtime conditional-if statement whether ```processRow()``` or ```processBatch``` called Do you mean these three features should be moved to ```InMemoryTableScanExec```? While I could do, my understanding is that these features would be implemented by methods in a singleton ```InMemoryTableScanExec```. Do you have any other good ideas to implement these features in ```InMemoryTableScanExec```? --- 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-14098][SQL] Generate Java code that get...
Github user davies commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-216953038 In high level, I think most of the implementation should be done in InMemoryTableScanExec, not in WholeStageCodegenExec. --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-216834259 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-216834262 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57749/ 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-216833964 **[Test build #57749 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57749/consoleFull)** for PR 11956 at commit [`87db6ba`](https://github.com/apache/spark/commit/87db6ba83bfdc322b9415a7e4f03da0b0cfc8bdf). * 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-216819530 **[Test build #57749 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57749/consoleFull)** for PR 11956 at commit [`87db6ba`](https://github.com/apache/spark/commit/87db6ba83bfdc322b9415a7e4f03da0b0cfc8bdf). --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-216811713 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 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-216811716 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57740/ 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-216811561 **[Test build #57740 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57740/consoleFull)** for PR 11956 at commit [`54eb492`](https://github.com/apache/spark/commit/54eb492415f386bf3deb3ec1a8d9316d9a084c4d). * This patch **fails Spark unit 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-216785078 **[Test build #57740 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57740/consoleFull)** for PR 11956 at commit [`54eb492`](https://github.com/apache/spark/commit/54eb492415f386bf3deb3ec1a8d9316d9a084c4d). --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-215976979 @davies for ease of review, it would be good to split this PR into two. One is only for float and double primitive types. The other is for other primitive types. The first one focuses on PassThrough compression type in Dataframe cache. The latter focuses on other compression types in Dataframe cache. I estimated that the size of each of PR is about 900 lines. Would it be possible to let us know If there are any issues? --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-215696918 @davies I love your idea to use Apache Arrow or other in-memory format instead of ```Array[Byte]``` for DataFrame cache. That is what I want to create a JIRA entry in near future. This idea is related to Part 1 (get data from columnar storage for DataFrame cache under ```org.apache.spark.sql.execution.columnar```) in this PR. Even if we use any in-memory format, an interface to code generation would be ColumnVector. As you can see, we already succeeded to wrap ```Array[Byte]``` and Parquet representation by using methods such as ```getInt()``` in ```ColumnVector```. In this PR, can we target Part 2 (codegen under```org.apache.spark.sql.execution```) of this PR for 2.0 at least? This is because Part 2 could accept any in-memory representation thru ```ColumnVector```. Part 2 is very small (less than 200 lines) after dropping code for Part 1. Thus, it is easy to review. --- 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-14098][SQL] Generate Java code that get...
Github user davies commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-215530899 @kiszk Can we target this for 2.1? We may switch to use Apache Arrow or other in-memory representation instead of current one. --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-215524446 We can shrink the size of this PR (reducing the number of lines from 1800 to 700) by dropping test suites (1450 lines) and a benchmark program (250 lines). Is it better for ease of your review? --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-215365815 ping @davies --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-214593726 @rxin, would it be possible to review this PR, too? Especially, for decompression part that you originally wrote. --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213881135 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213881140 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56827/ 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213881027 **[Test build #56827 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56827/consoleFull)** for PR 11956 at commit [`cd8855f`](https://github.com/apache/spark/commit/cd8855f963d180ff9aec07509b192633ebe8856b). * 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213872164 **[Test build #56827 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56827/consoleFull)** for PR 11956 at commit [`cd8855f`](https://github.com/apache/spark/commit/cd8855f963d180ff9aec07509b192633ebe8856b). --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213857098 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 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213857099 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56817/ 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213857070 **[Test build #56817 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56817/consoleFull)** for PR 11956 at commit [`2abd436`](https://github.com/apache/spark/commit/2abd436985bf556bb0f53c9f7967e50f3781465e). * This patch **fails Spark unit tests**. * This patch **does not merge 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213855047 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56819/ 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 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213855045 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213854956 **[Test build #56819 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56819/consoleFull)** for PR 11956 at commit [`f09ed72`](https://github.com/apache/spark/commit/f09ed721e3627772a89cea6095b5e87965c5ba60). * This patch **fails Spark unit 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213851558 **[Test build #56819 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56819/consoleFull)** for PR 11956 at commit [`f09ed72`](https://github.com/apache/spark/commit/f09ed721e3627772a89cea6095b5e87965c5ba60). --- 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213849521 **[Test build #56817 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56817/consoleFull)** for PR 11956 at commit [`2abd436`](https://github.com/apache/spark/commit/2abd436985bf556bb0f53c9f7967e50f3781465e). --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213744591 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 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213744592 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56801/ 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213744589 **[Test build #56801 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56801/consoleFull)** for PR 11956 at commit [`afccb66`](https://github.com/apache/spark/commit/afccb6637e064a4c072091b637d38c57a2fe0c9b). * This patch **fails to build**. * 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213744416 **[Test build #56801 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56801/consoleFull)** for PR 11956 at commit [`afccb66`](https://github.com/apache/spark/commit/afccb6637e064a4c072091b637d38c57a2fe0c9b). --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213417886 @davies, thank you for your great suggestion. Now, I simplified changes in ```WholeStageCodeGen.scala' by moving a logic regarding ```CachedBatch``` to other files. The total number of increased lines is reduced from 115 to 95.The generated code is also simplified since variables regarding ```CachedBatch``` are moved away. The total number of generated code is also reduced from 150 to 139. I updated generated code and performance result in this PR. --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213412679 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213412684 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56696/ 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213412355 **[Test build #56696 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56696/consoleFull)** for PR 11956 at commit [`ba4f929`](https://github.com/apache/spark/commit/ba4f929b37c52bf404a92bb4002e889ea2a0a2cc). * 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213381823 **[Test build #56696 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56696/consoleFull)** for PR 11956 at commit [`ba4f929`](https://github.com/apache/spark/commit/ba4f929b37c52bf404a92bb4002e889ea2a0a2cc). --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213301701 I know this is huge. Sorry for that. There are three reasons. One is to include benchmark. The other is include test cases. The last one is to newly introduce decompress() method for each of compression algorithm. Is it better to split this into two parts: decompress() and code generation? Also, I will try to reduce changes into WholeStageCodegen by moving some of them into other files such as InMemoryTableScan and GenerateColumnAccessor. --- 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-14098][SQL] Generate Java code that get...
Github user davies commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213299060 This PR is huge, I may not have enough time to review this. Just saw that there is lots of changes in WholeStageCodegen, is it possible to avoid most of them (simplify the logic or move it to InMemoryTableScan)? --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212971005 @davies , would it be possible to review this PR? --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212955113 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212955118 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56537/ 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212954704 **[Test build #56537 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56537/consoleFull)** for PR 11956 at commit [`4bc32e8`](https://github.com/apache/spark/commit/4bc32e894c8024a175db91c82272ff11ca2f9b27). * This patch passes all tests. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `case class CachedBatch(numRows: Int, buffers: Array[Array[Byte]], stats: InternalRow) ` --- 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212915500 **[Test build #56537 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56537/consoleFull)** for PR 11956 at commit [`4bc32e8`](https://github.com/apache/spark/commit/4bc32e894c8024a175db91c82272ff11ca2f9b27). --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212886235 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212885964 **[Test build #56526 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56526/consoleFull)** for PR 11956 at commit [`2735f5f`](https://github.com/apache/spark/commit/2735f5f922165d0a3acad28753a6cb1391fed664). * This patch **fails Spark unit tests**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `case class CachedBatch(numRows: Int, buffers: Array[Array[Byte]], stats: InternalRow) ` --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212886240 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56526/ 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212858662 **[Test build #56526 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56526/consoleFull)** for PR 11956 at commit [`2735f5f`](https://github.com/apache/spark/commit/2735f5f922165d0a3acad28753a6cb1391fed664). --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212422755 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 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212422758 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56348/ 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212422286 **[Test build #56348 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56348/consoleFull)** for PR 11956 at commit [`8212e20`](https://github.com/apache/spark/commit/8212e2008e2a7dcaad045477bc0e4fbb8f6d9897). * This patch **fails from timeout after a configured wait of \`250m\`**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `class DataFrameCacheSuite extends QueryTest with SharedSQLContext ` --- 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212341265 **[Test build #56348 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56348/consoleFull)** for PR 11956 at commit [`8212e20`](https://github.com/apache/spark/commit/8212e2008e2a7dcaad045477bc0e4fbb8f6d9897). --- 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-14098][SQL] Generate Java code that get...
Github user kiszk commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212340486 Jenkins, retest 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 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212163092 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56255/ 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212162726 **[Test build #56255 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56255/consoleFull)** for PR 11956 at commit [`8212e20`](https://github.com/apache/spark/commit/8212e2008e2a7dcaad045477bc0e4fbb8f6d9897). * This patch **fails from timeout after a configured wait of \`250m\`**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `class DataFrameCacheSuite extends QueryTest with SharedSQLContext ` --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212163086 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 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212126159 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56237/ 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 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212126150 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212125663 **[Test build #56237 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56237/consoleFull)** for PR 11956 at commit [`c47a492`](https://github.com/apache/spark/commit/c47a492f895b54782463405a122a84fff1d36cf8). * This patch **fails from timeout after a configured wait of \`250m\`**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `class DataFrameCacheSuite extends QueryTest with SharedSQLContext ` --- 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212068656 **[Test build #56255 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56255/consoleFull)** for PR 11956 at commit [`8212e20`](https://github.com/apache/spark/commit/8212e2008e2a7dcaad045477bc0e4fbb8f6d9897). --- 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212011800 **[Test build #56237 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56237/consoleFull)** for PR 11956 at commit [`c47a492`](https://github.com/apache/spark/commit/c47a492f895b54782463405a122a84fff1d36cf8). --- 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212009452 **[Test build #56236 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56236/consoleFull)** for PR 11956 at commit [`5075cd2`](https://github.com/apache/spark/commit/5075cd2c9e2eb444ef76b628003940555f9dd336). * This patch **fails Scala style 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212009462 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56236/ 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 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212009459 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-212009113 **[Test build #56236 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56236/consoleFull)** for PR 11956 at commit [`5075cd2`](https://github.com/apache/spark/commit/5075cd2c9e2eb444ef76b628003940555f9dd336). --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-211858281 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 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-211858288 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56203/ 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-211857940 **[Test build #56203 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56203/consoleFull)** for PR 11956 at commit [`23b6ae1`](https://github.com/apache/spark/commit/23b6ae1e368d317d9c15afb31acd878d13119e43). * This patch **fails Spark unit 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-211769694 **[Test build #56203 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56203/consoleFull)** for PR 11956 at commit [`23b6ae1`](https://github.com/apache/spark/commit/23b6ae1e368d317d9c15afb31acd878d13119e43). --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-211594470 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56093/ 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 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-211594468 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-211594323 **[Test build #56093 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56093/consoleFull)** for PR 11956 at commit [`6cfa2ff`](https://github.com/apache/spark/commit/6cfa2ffa19bd7c53f98f205455a8a1be78bf4ff8). * This patch **fails from timeout after a configured wait of \`250m\`**. * 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-211487918 **[Test build #56093 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56093/consoleFull)** for PR 11956 at commit [`6cfa2ff`](https://github.com/apache/spark/commit/6cfa2ffa19bd7c53f98f205455a8a1be78bf4ff8). --- 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-211066154 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56058/ 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 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-14098][SQL] Generate Java code that get...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-211065793 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 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-14098][SQL] Generate Java code that get...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-211063014 **[Test build #56058 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56058/consoleFull)** for PR 11956 at commit [`fe2043c`](https://github.com/apache/spark/commit/fe2043cefc7cb15df433c70ec11b35e5012e2919). * This patch **fails Spark unit 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