[GitHub] [spark] SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552197822 **[Test build #113543 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552197606 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
SparkQA removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552197183 **[Test build #113542 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals URL: https://github.com/apache/spark/pull/26438#issuecomment-552197618 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552197606 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552197601 **[Test build #113542 has

[GitHub] [spark] AmplabJenkins commented on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals URL: https://github.com/apache/spark/pull/26438#issuecomment-552197622 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals URL: https://github.com/apache/spark/pull/26438#issuecomment-552197618 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins removed a comment on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals URL: https://github.com/apache/spark/pull/26438#issuecomment-552197622 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552197603 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552197603 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA removed a comment on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals

2019-11-10 Thread GitBox
SparkQA removed a comment on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals URL: https://github.com/apache/spark/pull/26438#issuecomment-552181610 **[Test build #113535 has

[GitHub] [spark] SparkQA commented on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals

2019-11-10 Thread GitBox
SparkQA commented on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals URL: https://github.com/apache/spark/pull/26438#issuecomment-552197504 **[Test build #113535 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552197280 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552197277 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552197277 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552197280 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552197183 **[Test build #113542 has

[GitHub] [spark] ulysses-you commented on a change in pull request #26391: [SPARK-29749][SQL] Add ParquetScan statistics

2019-11-10 Thread GitBox
ulysses-you commented on a change in pull request #26391: [SPARK-29749][SQL] Add ParquetScan statistics URL: https://github.com/apache/spark/pull/26391#discussion_r344493522 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetUtils.scala

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552196944 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552196941 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
SparkQA removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552196486 **[Test build #113541 has

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552196941 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552196933 **[Test build #113541 has

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552196944 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552196624 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552196621 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552196621 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552196624 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552196486 **[Test build #113541 has

[GitHub] [spark] maropu commented on issue #26458: [SPARK-29821] Allow calling non-aggregate SQL functions with column name

2019-11-10 Thread GitBox
maropu commented on issue #26458: [SPARK-29821] Allow calling non-aggregate SQL functions with column name URL: https://github.com/apache/spark/pull/26458#issuecomment-552192167 IIRC we don't actively add an interface for string column names in functions. plz use `stringExpr` instead. cc:

[GitHub] [spark] AmplabJenkins commented on issue #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite URL: https://github.com/apache/spark/pull/26459#issuecomment-552191853 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite URL: https://github.com/apache/spark/pull/26459#issuecomment-552191850 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite URL: https://github.com/apache/spark/pull/26459#issuecomment-552191850 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite URL: https://github.com/apache/spark/pull/26459#issuecomment-552191853 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] SparkQA commented on issue #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite

2019-11-10 Thread GitBox
SparkQA commented on issue #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite URL: https://github.com/apache/spark/pull/26459#issuecomment-552191755 **[Test build #113540 has

[GitHub] [spark] maropu commented on issue #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite

2019-11-10 Thread GitBox
maropu commented on issue #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite URL: https://github.com/apache/spark/pull/26459#issuecomment-552191508 cc: @wangyum @dongjoon-hyun @HyukjinKwon

[GitHub] [spark] maropu opened a new pull request #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite

2019-11-10 Thread GitBox
maropu opened a new pull request #26459: [SPARK-29825][SQL][TESTS] Add join conditions in join-related tests of SQLQueryTestSuite URL: https://github.com/apache/spark/pull/26459 ### What changes were proposed in this pull request? This pr is to add missing join conditions in

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552190870 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552190870 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
SparkQA removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552190575 **[Test build #113539 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552190867 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552190867 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552190865 **[Test build #113539 has

[GitHub] [spark] SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552190575 **[Test build #113539 has

[GitHub] [spark] yaooqinn commented on a change in pull request #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
yaooqinn commented on a change in pull request #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#discussion_r344489093 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552190075 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#issuecomment-552190058 Test PASSed. Refer to this link for build results (access rights

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552190079 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] yaooqinn commented on a change in pull request #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
yaooqinn commented on a change in pull request #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#discussion_r344489093 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#issuecomment-552190058 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552190075 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552190079 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#issuecomment-552190056 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#issuecomment-552190056 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
SparkQA commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#issuecomment-552189946 **[Test build #113538 has

[GitHub] [spark] AngersZhuuuu commented on a change in pull request #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
AngersZh commented on a change in pull request #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#discussion_r344488915 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552189695 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
SparkQA removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552189377 **[Test build #113537 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552189692 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552189692 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552189689 **[Test build #113537 has

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552189695 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552189483 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552189487 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552189483 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552189487 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
SparkQA commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#issuecomment-552189372 **[Test build #113536 has

[GitHub] [spark] SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals

2019-11-10 Thread GitBox
SparkQA commented on issue #26359: [SPARK-29713][SQL] Support Interval Unit Abbreviations in Interval Literals URL: https://github.com/apache/spark/pull/26359#issuecomment-552189377 **[Test build #113537 has

[GitHub] [spark] AmplabJenkins commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#issuecomment-552188891 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#issuecomment-55218 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#issuecomment-55218 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#issuecomment-552188891 Test PASSed. Refer to this link for build results (access rights

[GitHub] [spark] AmplabJenkins removed a comment on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#issuecomment-550123593 Can one of the admins verify this patch?

[GitHub] [spark] maropu commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
maropu commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#issuecomment-552188798 ok to test This is

[GitHub] [spark] maropu commented on a change in pull request #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
maropu commented on a change in pull request #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#discussion_r344488185 ## File path:

[GitHub] [spark] maropu commented on a change in pull request #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
maropu commented on a change in pull request #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#discussion_r344488185 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals URL: https://github.com/apache/spark/pull/26438#issuecomment-552188484 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals URL: https://github.com/apache/spark/pull/26438#issuecomment-552188483 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins removed a comment on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals URL: https://github.com/apache/spark/pull/26438#issuecomment-552188484 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals URL: https://github.com/apache/spark/pull/26438#issuecomment-552188483 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA removed a comment on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals

2019-11-10 Thread GitBox
SparkQA removed a comment on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals URL: https://github.com/apache/spark/pull/26438#issuecomment-552173790 **[Test build #113533 has

[GitHub] [spark] SparkQA commented on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals

2019-11-10 Thread GitBox
SparkQA commented on issue #26438: [SPARK-29408][SQL] Support `-` before `interval` in interval literals URL: https://github.com/apache/spark/pull/26438#issuecomment-552188356 **[Test build #113533 has

[GitHub] [spark] AmplabJenkins commented on issue #26455: [SPARK-29819][SQL] Introduce an enum for interval units

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26455: [SPARK-29819][SQL] Introduce an enum for interval units URL: https://github.com/apache/spark/pull/26455#issuecomment-552188301 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins removed a comment on issue #26455: [SPARK-29819][SQL] Introduce an enum for interval units

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26455: [SPARK-29819][SQL] Introduce an enum for interval units URL: https://github.com/apache/spark/pull/26455#issuecomment-552188301 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins removed a comment on issue #26455: [SPARK-29819][SQL] Introduce an enum for interval units

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26455: [SPARK-29819][SQL] Introduce an enum for interval units URL: https://github.com/apache/spark/pull/26455#issuecomment-552188304 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26455: [SPARK-29819][SQL] Introduce an enum for interval units

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26455: [SPARK-29819][SQL] Introduce an enum for interval units URL: https://github.com/apache/spark/pull/26455#issuecomment-552188304 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #26455: [SPARK-29819][SQL] Introduce an enum for interval units

2019-11-10 Thread GitBox
SparkQA removed a comment on issue #26455: [SPARK-29819][SQL] Introduce an enum for interval units URL: https://github.com/apache/spark/pull/26455#issuecomment-552173781 **[Test build #113531 has

[GitHub] [spark] SparkQA commented on issue #26455: [SPARK-29819][SQL] Introduce an enum for interval units

2019-11-10 Thread GitBox
SparkQA commented on issue #26455: [SPARK-29819][SQL] Introduce an enum for interval units URL: https://github.com/apache/spark/pull/26455#issuecomment-552188185 **[Test build #113531 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/113531/testReport)**

[GitHub] [spark] AngersZhuuuu commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql`

2019-11-10 Thread GitBox
AngersZh commented on issue #26406: [SPARK-29145][SQL][FOLLOW-UP] Move tests from `SubquerySuite` to `subquery/in-subquery/in-joins.sql` URL: https://github.com/apache/spark/pull/26406#issuecomment-552188011 > Can you remove the test of `SubquerySuite` that you added in the previous

[GitHub] [spark] AmplabJenkins removed a comment on issue #26446: [SPARK-29393][SQL] Add `make_interval` function

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26446: [SPARK-29393][SQL] Add `make_interval` function URL: https://github.com/apache/spark/pull/26446#issuecomment-552187827 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins removed a comment on issue #26446: [SPARK-29393][SQL] Add `make_interval` function

2019-11-10 Thread GitBox
AmplabJenkins removed a comment on issue #26446: [SPARK-29393][SQL] Add `make_interval` function URL: https://github.com/apache/spark/pull/26446#issuecomment-552187830 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26446: [SPARK-29393][SQL] Add `make_interval` function

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26446: [SPARK-29393][SQL] Add `make_interval` function URL: https://github.com/apache/spark/pull/26446#issuecomment-552187827 Merged build finished. Test PASSed. This is an automated message

[GitHub] [spark] AmplabJenkins commented on issue #26446: [SPARK-29393][SQL] Add `make_interval` function

2019-11-10 Thread GitBox
AmplabJenkins commented on issue #26446: [SPARK-29393][SQL] Add `make_interval` function URL: https://github.com/apache/spark/pull/26446#issuecomment-552187830 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #26446: [SPARK-29393][SQL] Add `make_interval` function

2019-11-10 Thread GitBox
SparkQA removed a comment on issue #26446: [SPARK-29393][SQL] Add `make_interval` function URL: https://github.com/apache/spark/pull/26446#issuecomment-552173255 **[Test build #113530 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/113530/testReport)** for

[GitHub] [spark] SparkQA commented on issue #26446: [SPARK-29393][SQL] Add `make_interval` function

2019-11-10 Thread GitBox
SparkQA commented on issue #26446: [SPARK-29393][SQL] Add `make_interval` function URL: https://github.com/apache/spark/pull/26446#issuecomment-552187718 **[Test build #113530 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/113530/testReport)** for PR

[GitHub] [spark] maropu commented on issue #26406: [WIP][SPARK-29145][SQL][FOLLOW-UP] Port in-subquery-on-join-condition.sql

2019-11-10 Thread GitBox
maropu commented on issue #26406: [WIP][SPARK-29145][SQL][FOLLOW-UP] Port in-subquery-on-join-condition.sql URL: https://github.com/apache/spark/pull/26406#issuecomment-552187091 Can you remove the test of `SubquerySuite` that you added in the previous pr?

[GitHub] [spark] maropu edited a comment on issue #26420: [SPARK-27986][SQL] Support ANSI SQL filter predicate for aggregate expression.

2019-11-10 Thread GitBox
maropu edited a comment on issue #26420: [SPARK-27986][SQL] Support ANSI SQL filter predicate for aggregate expression. URL: https://github.com/apache/spark/pull/26420#issuecomment-552186656 I meant we might not need to modify the physical plans for aggregates (e.g., HashAggregateExec).

[GitHub] [spark] maropu commented on issue #26420: [SPARK-27986][SQL] Support ANSI SQL filter predicate for aggregate expression.

2019-11-10 Thread GitBox
maropu commented on issue #26420: [SPARK-27986][SQL] Support ANSI SQL filter predicate for aggregate expression. URL: https://github.com/apache/spark/pull/26420#issuecomment-552186656 I meant we might not need to modify the physical plans for aggregates (e.g., HashAggregateExec). Instead,

[GitHub] [spark] AngersZhuuuu commented on issue #26406: [WIP][SPARK-29145][SQL][FOLLOW-UP] Port in-subquery-on-join-condition.sql

2019-11-10 Thread GitBox
AngersZh commented on issue #26406: [WIP][SPARK-29145][SQL][FOLLOW-UP] Port in-subquery-on-join-condition.sql URL: https://github.com/apache/spark/pull/26406#issuecomment-552186679 > Yea, you need to nothing for that. btw, still `WIP` in this pr? Remove `WIP`

<    1   2   3   4   5   6   7   >