[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/19188 LGTM --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/19188 Merging to master. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81745/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81745 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81745/testReport)** for PR 19188 at commit [`b543e71`](https://github.com/apache/spark/commit/b543e710dae79da33a9334d5bbe4bb474a44b39c). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81745 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81745/testReport)** for PR 19188 at commit [`b543e71`](https://github.com/apache/spark/commit/b543e710dae79da33a9334d5bbe4bb474a44b39c). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81740/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81740 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81740/testReport)** for PR 19188 at commit [`c957e47`](https://github.com/apache/spark/commit/c957e47d0f4723e62cb0b8a675703a75e14a37c8). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81740 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81740/testReport)** for PR 19188 at commit [`c957e47`](https://github.com/apache/spark/commit/c957e47d0f4723e62cb0b8a675703a75e14a37c8). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user maropu commented on the issue: https://github.com/apache/spark/pull/19188 @gatorsmile could u check? Thanks~ --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81703/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81703 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81703/testReport)** for PR 19188 at commit [`cc11163`](https://github.com/apache/spark/commit/cc111630cd9311dc71ec50dd7915673e95dd520e). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81702/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81702 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81702/testReport)** for PR 19188 at commit [`be1a199`](https://github.com/apache/spark/commit/be1a1993f1e292793946e73b1e9b3f6d66f73e63). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81701/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81701 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81701/testReport)** for PR 19188 at commit [`12767bc`](https://github.com/apache/spark/commit/12767bcbf7c763451b23f3726caa972d699325d4). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81703 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81703/testReport)** for PR 19188 at commit [`cc11163`](https://github.com/apache/spark/commit/cc111630cd9311dc71ec50dd7915673e95dd520e). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user maropu commented on the issue: https://github.com/apache/spark/pull/19188 hmmm. ok, currently, we have two options only, so I feel okay to keep it now. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user viirya commented on the issue: https://github.com/apache/spark/pull/19188 But looks like current `TPCDSQueryBenchmarkArguments` is not good to test individually... --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user viirya commented on the issue: https://github.com/apache/spark/pull/19188 Yeah, it's good if you can add one. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user maropu commented on the issue: https://github.com/apache/spark/pull/19188 better to add tests for `TPCDSQueryBenchmarkArguments`? we have tests for `SparkSubmitArguments` in `SparkSubmitSuite` though. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user viirya commented on the issue: https://github.com/apache/spark/pull/19188 LGTM with few minor comments. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user maropu commented on the issue: https://github.com/apache/spark/pull/19188 Also, I manually checked if it worked. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81702 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81702/testReport)** for PR 19188 at commit [`be1a199`](https://github.com/apache/spark/commit/be1a1993f1e292793946e73b1e9b3f6d66f73e63). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81701 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81701/testReport)** for PR 19188 at commit [`12767bc`](https://github.com/apache/spark/commit/12767bcbf7c763451b23f3726caa972d699325d4). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user maropu commented on the issue: https://github.com/apache/spark/pull/19188 ok, I'll update in a day! Thanks! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/19188 This has been merged. @maropu Could you update this PR? Thanks! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81637/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81637 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81637/testReport)** for PR 19188 at commit [`322c335`](https://github.com/apache/spark/commit/322c335a027edc8c6705b72fd76f00e6933a987f). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81637 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81637/testReport)** for PR 19188 at commit [`322c335`](https://github.com/apache/spark/commit/322c335a027edc8c6705b72fd76f00e6933a987f). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81633/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81633 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81633/testReport)** for PR 19188 at commit [`4665e85`](https://github.com/apache/spark/commit/4665e852f82933be02dd93990e858dba550d73a4). * This patch **fails Scala style tests**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * ` case class TpcdsQueries(spark: SparkSession, queries: Seq[String], dataLocation: String) ` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19188 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #19188: [SPARK-21973][SQL] Add an new option to filter queries i...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19188 **[Test build #81633 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81633/testReport)** for PR 19188 at commit [`4665e85`](https://github.com/apache/spark/commit/4665e852f82933be02dd93990e858dba550d73a4). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org