[GitHub] [spark] AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612573957 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.c

[GitHub] [spark] SparkQA removed a comment on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check.

2020-04-12 Thread GitBox
SparkQA removed a comment on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612572858 **[Test build #121141 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12114

[GitHub] [spark] AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612573954 Merged build finished. Test FAILed. This is

[GitHub] [spark] AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612573961 Merged build finished. Test FAILed. This is

[GitHub] [spark] SparkQA commented on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check.

2020-04-12 Thread GitBox
SparkQA commented on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612573942 **[Test build #121140 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121140/testR

[GitHub] [spark] SparkQA removed a comment on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check.

2020-04-12 Thread GitBox
SparkQA removed a comment on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612569454 **[Test build #121140 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12114

[GitHub] [spark] SparkQA commented on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check.

2020-04-12 Thread GitBox
SparkQA commented on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612573941 **[Test build #121141 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121141/testR

[GitHub] [spark] AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612573963 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.c

[GitHub] [spark] AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612573961 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612573954 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612573957 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://

[GitHub] [spark] AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][Test] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612573963 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://

[GitHub] [spark] sarutak commented on a change in pull request #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen

2020-04-12 Thread GitBox
sarutak commented on a change in pull request #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen URL: https://github.com/apache/spark/pull/28184#discussion_r407161504 ## File path: sql/catalyst/src/main/scala/org/apa

[GitHub] [spark] SparkQA commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
SparkQA commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612578223 **[Test build #121142 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullReq

[GitHub] [spark] AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612578360 Test PASSed. Refer to this link for build results (access rights to CI server need

[GitHub] [spark] AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612578358 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612578358 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612578360 Test PASSed. Refer to this link for build results (access rights to CI ser

[GitHub] [spark] AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612579942 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
SparkQA commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612579928 **[Test build #121142 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRe

[GitHub] [spark] AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612579945 Test FAILed. Refer to this link for build results (access rights to CI server need

[GitHub] [spark] AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612579942 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
SparkQA removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612578223 **[Test build #121142 has started](https://amplab.cs.berkeley.edu/jenkins/job/Spar

[GitHub] [spark] AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612579945 Test FAILed. Refer to this link for build results (access rights to CI ser

[GitHub] [spark] SparkQA commented on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen

2020-04-12 Thread GitBox
SparkQA commented on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen URL: https://github.com/apache/spark/pull/28184#issuecomment-612581029 **[Test build #121143 has started](https://amplab.cs.berkeley.edu/jenkins/jo

[GitHub] [spark] AmplabJenkins commented on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen URL: https://github.com/apache/spark/pull/28184#issuecomment-612581167 Test PASSed. Refer to this link for build results (access rights to

[GitHub] [spark] AmplabJenkins commented on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen URL: https://github.com/apache/spark/pull/28184#issuecomment-612581163 Merged build finished. Test PASSed. --

[GitHub] [spark] AmplabJenkins removed a comment on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen URL: https://github.com/apache/spark/pull/28184#issuecomment-612581163 Merged build finished. Test PASSed. --

[GitHub] [spark] AmplabJenkins removed a comment on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen URL: https://github.com/apache/spark/pull/28184#issuecomment-612581167 Test PASSed. Refer to this link for build results (access ri

[GitHub] [spark] SparkQA commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
SparkQA commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612584048 **[Test build #121144 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullReq

[GitHub] [spark] AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612584166 Test PASSed. Refer to this link for build results (access rights to CI server need

[GitHub] [spark] AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612584164 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612584164 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612584166 Test PASSed. Refer to this link for build results (access rights to CI ser

[GitHub] [spark] SparkQA commented on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen

2020-04-12 Thread GitBox
SparkQA commented on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen URL: https://github.com/apache/spark/pull/28184#issuecomment-612607708 **[Test build #121143 has finished](https://amplab.cs.berkeley.edu/jenkins/j

[GitHub] [spark] AmplabJenkins commented on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen URL: https://github.com/apache/spark/pull/28184#issuecomment-612607903 Test PASSed. Refer to this link for build results (access rights to

[GitHub] [spark] AmplabJenkins commented on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen URL: https://github.com/apache/spark/pull/28184#issuecomment-612607898 Merged build finished. Test PASSed. --

[GitHub] [spark] SparkQA removed a comment on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen

2020-04-12 Thread GitBox
SparkQA removed a comment on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen URL: https://github.com/apache/spark/pull/28184#issuecomment-612581029 **[Test build #121143 has started](https://amplab.cs.berkeley.edu/je

[GitHub] [spark] AmplabJenkins removed a comment on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen URL: https://github.com/apache/spark/pull/28184#issuecomment-612607903 Test PASSed. Refer to this link for build results (access ri

[GitHub] [spark] AmplabJenkins removed a comment on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28184: [SPARK-31416][SQL] Check more strictly that a field name can be used as a valid Java identifier for codegen URL: https://github.com/apache/spark/pull/28184#issuecomment-612607898 Merged build finished. Test PASSed. --

[GitHub] [spark] SparkQA commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
SparkQA commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612611176 **[Test build #121144 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRe

[GitHub] [spark] SparkQA removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
SparkQA removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612584048 **[Test build #121144 has started](https://amplab.cs.berkeley.edu/jenkins/job/Spar

[GitHub] [spark] AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612611410 Test PASSed. Refer to this link for build results (access rights to CI server need

[GitHub] [spark] AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612611409 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612611409 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28164: [SPARK-31393][SQL] Show the correct alias in a more elegant way that override the prettyName URL: https://github.com/apache/spark/pull/28164#issuecomment-612611410 Test PASSed. Refer to this link for build results (access rights to CI ser

[GitHub] [spark] beliefer commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check.

2020-04-12 Thread GitBox
beliefer commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612612854 retest this please This is an automated message f

[GitHub] [spark] SparkQA commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check.

2020-04-12 Thread GitBox
SparkQA commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612613510 **[Test build #121145 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121145/testRe

[GitHub] [spark] AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612613676 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.

[GitHub] [spark] AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612613674 Merged build finished. Test PASSed. This is

[GitHub] [spark] AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612613676 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:/

[GitHub] [spark] AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612613674 Merged build finished. Test PASSed.

[GitHub] [spark] Ngone51 opened a new pull request #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset

2020-04-12 Thread GitBox
Ngone51 opened a new pull request #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset URL: https://github.com/apache/spark/pull/28195 ### What changes were proposed in this pull request? Make `UnsafeKVExternalSorter` also respect `U

[GitHub] [spark] SparkQA commented on issue #27066: [SPARK-31317][SQL] Add withField method to Column class

2020-04-12 Thread GitBox
SparkQA commented on issue #27066: [SPARK-31317][SQL] Add withField method to Column class URL: https://github.com/apache/spark/pull/27066#issuecomment-612628076 **[Test build #121146 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121146/testReport)** for

[GitHub] [spark] AmplabJenkins commented on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset URL: https://github.com/apache/spark/pull/28195#issuecomment-612628244 Merged build finished. Test PASSed. --

[GitHub] [spark] AmplabJenkins commented on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset URL: https://github.com/apache/spark/pull/28195#issuecomment-612628250 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset

2020-04-12 Thread GitBox
SparkQA commented on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset URL: https://github.com/apache/spark/pull/28195#issuecomment-612629291 **[Test build #121147 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBu

[GitHub] [spark] AmplabJenkins removed a comment on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset URL: https://github.com/apache/spark/pull/28195#issuecomment-612628250 Test PASSed. Refer to this link for build results (access rights to CI server ne

[GitHub] [spark] AmplabJenkins removed a comment on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset URL: https://github.com/apache/spark/pull/28195#issuecomment-612628244 Merged build finished. Test PASSed. --

[GitHub] [spark] AmplabJenkins commented on issue #27701: [SPARK-30953][SQL] InsertAdaptiveSparkPlan should apply AQE on child plan of write commands

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #27701: [SPARK-30953][SQL] InsertAdaptiveSparkPlan should apply AQE on child plan of write commands URL: https://github.com/apache/spark/pull/27701#issuecomment-612635691 Test PASSed. Refer to this link for build results (access rights to CI server needed

[GitHub] [spark] AmplabJenkins removed a comment on issue #27701: [SPARK-30953][SQL] InsertAdaptiveSparkPlan should apply AQE on child plan of write commands

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #27701: [SPARK-30953][SQL] InsertAdaptiveSparkPlan should apply AQE on child plan of write commands URL: https://github.com/apache/spark/pull/27701#issuecomment-612635691 Test PASSed. Refer to this link for build results (access rights to CI serve

[GitHub] [spark] AmplabJenkins commented on issue #27701: [SPARK-30953][SQL] InsertAdaptiveSparkPlan should apply AQE on child plan of write commands

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #27701: [SPARK-30953][SQL] InsertAdaptiveSparkPlan should apply AQE on child plan of write commands URL: https://github.com/apache/spark/pull/27701#issuecomment-612635685 Merged build finished. Test PASSed. --

[GitHub] [spark] AmplabJenkins removed a comment on issue #27701: [SPARK-30953][SQL] InsertAdaptiveSparkPlan should apply AQE on child plan of write commands

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #27701: [SPARK-30953][SQL] InsertAdaptiveSparkPlan should apply AQE on child plan of write commands URL: https://github.com/apache/spark/pull/27701#issuecomment-612635685 Merged build finished. Test PASSed. --

[GitHub] [spark] SparkQA commented on issue #27701: [SPARK-30953][SQL] InsertAdaptiveSparkPlan should apply AQE on child plan of write commands

2020-04-12 Thread GitBox
SparkQA commented on issue #27701: [SPARK-30953][SQL] InsertAdaptiveSparkPlan should apply AQE on child plan of write commands URL: https://github.com/apache/spark/pull/27701#issuecomment-612636578 **[Test build #121148 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullReque

[GitHub] [spark] SparkQA commented on issue #28177: [SPARK-31407][SQL][TEST] Fix hive/SQLQuerySuite.derived from Hive query file: drop_database_removes_partition_dirs.q

2020-04-12 Thread GitBox
SparkQA commented on issue #28177: [SPARK-31407][SQL][TEST] Fix hive/SQLQuerySuite.derived from Hive query file: drop_database_removes_partition_dirs.q URL: https://github.com/apache/spark/pull/28177#issuecomment-612641606 **[Test build #121149 has started](https://amplab.cs.berkeley.edu/

[GitHub] [spark] AmplabJenkins commented on issue #28177: [SPARK-31407][SQL][TEST] Fix hive/SQLQuerySuite.derived from Hive query file: drop_database_removes_partition_dirs.q

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28177: [SPARK-31407][SQL][TEST] Fix hive/SQLQuerySuite.derived from Hive query file: drop_database_removes_partition_dirs.q URL: https://github.com/apache/spark/pull/28177#issuecomment-612641763 Test PASSed. Refer to this link for build results (access

[GitHub] [spark] AmplabJenkins commented on issue #28177: [SPARK-31407][SQL][TEST] Fix hive/SQLQuerySuite.derived from Hive query file: drop_database_removes_partition_dirs.q

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28177: [SPARK-31407][SQL][TEST] Fix hive/SQLQuerySuite.derived from Hive query file: drop_database_removes_partition_dirs.q URL: https://github.com/apache/spark/pull/28177#issuecomment-612641761 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #28177: [SPARK-31407][SQL][TEST] Fix hive/SQLQuerySuite.derived from Hive query file: drop_database_removes_partition_dirs.q

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28177: [SPARK-31407][SQL][TEST] Fix hive/SQLQuerySuite.derived from Hive query file: drop_database_removes_partition_dirs.q URL: https://github.com/apache/spark/pull/28177#issuecomment-612641761 Merged build finished. Test PASSed.

[GitHub] [spark] MaxGekk commented on issue #28189: [SPARK-31426][SQL] Fix regressions of toJavaTimestamp/fromJavaTimestamp

2020-04-12 Thread GitBox
MaxGekk commented on issue #28189: [SPARK-31426][SQL] Fix regressions of toJavaTimestamp/fromJavaTimestamp URL: https://github.com/apache/spark/pull/28189#issuecomment-612641776 @cloud-fan @HyukjinKwon @dongjoon-hyun Please, have a look at the PR. -

[GitHub] [spark] AmplabJenkins removed a comment on issue #28177: [SPARK-31407][SQL][TEST] Fix hive/SQLQuerySuite.derived from Hive query file: drop_database_removes_partition_dirs.q

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28177: [SPARK-31407][SQL][TEST] Fix hive/SQLQuerySuite.derived from Hive query file: drop_database_removes_partition_dirs.q URL: https://github.com/apache/spark/pull/28177#issuecomment-612641763 Test PASSed. Refer to this link for build results

[GitHub] [spark] SparkQA commented on issue #28041: [SPARK-30564][SQL] Improved extra new line and comment remove

2020-04-12 Thread GitBox
SparkQA commented on issue #28041: [SPARK-30564][SQL] Improved extra new line and comment remove URL: https://github.com/apache/spark/pull/28041#issuecomment-612642563 **[Test build #121150 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121150/testReport)**

[GitHub] [spark] AmplabJenkins commented on issue #28041: [SPARK-30564][SQL] Improved extra new line and comment remove

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28041: [SPARK-30564][SQL] Improved extra new line and comment remove URL: https://github.com/apache/spark/pull/28041#issuecomment-612642680 Merged build finished. Test PASSed. This is an auto

[GitHub] [spark] AmplabJenkins removed a comment on issue #28041: [SPARK-30564][SQL] Improved extra new line and comment remove

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28041: [SPARK-30564][SQL] Improved extra new line and comment remove URL: https://github.com/apache/spark/pull/28041#issuecomment-612642681 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab

[GitHub] [spark] AmplabJenkins commented on issue #28041: [SPARK-30564][SQL] Improved extra new line and comment remove

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28041: [SPARK-30564][SQL] Improved extra new line and comment remove URL: https://github.com/apache/spark/pull/28041#issuecomment-612642681 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berk

[GitHub] [spark] AmplabJenkins removed a comment on issue #28041: [SPARK-30564][SQL] Improved extra new line and comment remove

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28041: [SPARK-30564][SQL] Improved extra new line and comment remove URL: https://github.com/apache/spark/pull/28041#issuecomment-612642680 Merged build finished. Test PASSed. This is

[GitHub] [spark] MaxGekk commented on issue #28169: [SPARK-31398][SQL] Fix regression of loading dates before 1582 year by non-vectorized ORC reader.

2020-04-12 Thread GitBox
MaxGekk commented on issue #28169: [SPARK-31398][SQL] Fix regression of loading dates before 1582 year by non-vectorized ORC reader. URL: https://github.com/apache/spark/pull/28169#issuecomment-612643046 @cloud-fan @HyukjinKwon @dongjoon-hyun Please, review this PR. ---

[GitHub] [spark] peter-toth commented on issue #28041: [SPARK-30564][SQL] Improved extra new line and comment remove

2020-04-12 Thread GitBox
peter-toth commented on issue #28041: [SPARK-30564][SQL] Improved extra new line and comment remove URL: https://github.com/apache/spark/pull/28041#issuecomment-612643258 I looked into this issue a bit deeper. Let me share my thoughts: - I still don't think reverting the comment removal

[GitHub] [spark] MaxGekk commented on issue #28172: [SPARK-31402][SQL] Fix rebasing of BCE dates/timestamps

2020-04-12 Thread GitBox
MaxGekk commented on issue #28172: [SPARK-31402][SQL] Fix rebasing of BCE dates/timestamps URL: https://github.com/apache/spark/pull/28172#issuecomment-612643219 @cloud-fan @HyukjinKwon @dongjoon-hyun Please, review this PR.

[GitHub] [spark] peter-toth commented on a change in pull request #28041: [SPARK-30564][SQL] Improved extra new line and comment remove

2020-04-12 Thread GitBox
peter-toth commented on a change in pull request #28041: [SPARK-30564][SQL] Improved extra new line and comment remove URL: https://github.com/apache/spark/pull/28041#discussion_r407222956 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/CodeFormatterBenchmark.scala

[GitHub] [spark] peter-toth commented on issue #28083: [WIP][SPARK-30564][SQL] Use comment placeholders in HashAggregateExec

2020-04-12 Thread GitBox
peter-toth commented on issue #28083: [WIP][SPARK-30564][SQL] Use comment placeholders in HashAggregateExec URL: https://github.com/apache/spark/pull/28083#issuecomment-612643535 > Yes still WIP, didn't have time to work on this unfortunately. I will try to sort this out during the weekend

[GitHub] [spark] peter-toth edited a comment on issue #28083: [WIP][SPARK-30564][SQL] Use comment placeholders in HashAggregateExec

2020-04-12 Thread GitBox
peter-toth edited a comment on issue #28083: [WIP][SPARK-30564][SQL] Use comment placeholders in HashAggregateExec URL: https://github.com/apache/spark/pull/28083#issuecomment-612643535 > Yes still WIP, didn't have time to work on this unfortunately. I will try to sort this out during the

[GitHub] [spark] MaxGekk closed pull request #28091: [WIP][SQL] Optimize fromJavaDate() and toJavaDate()

2020-04-12 Thread GitBox
MaxGekk closed pull request #28091: [WIP][SQL] Optimize fromJavaDate() and toJavaDate() URL: https://github.com/apache/spark/pull/28091 This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [spark] MaxGekk commented on issue #28091: [WIP][SQL] Optimize fromJavaDate() and toJavaDate()

2020-04-12 Thread GitBox
MaxGekk commented on issue #28091: [WIP][SQL] Optimize fromJavaDate() and toJavaDate() URL: https://github.com/apache/spark/pull/28091#issuecomment-612644143 I am closing this PR because the final results look not so good: ```diff OpenJDK 64-Bit Server VM 1.8.0_242-8u242-b08-0ubuntu3~

[GitHub] [spark] SparkQA commented on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset

2020-04-12 Thread GitBox
SparkQA commented on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset URL: https://github.com/apache/spark/pull/28195#issuecomment-612649156 **[Test build #121147 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestB

[GitHub] [spark] SparkQA removed a comment on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset

2020-04-12 Thread GitBox
SparkQA removed a comment on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset URL: https://github.com/apache/spark/pull/28195#issuecomment-612629291 **[Test build #121147 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullR

[GitHub] [spark] AmplabJenkins removed a comment on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset URL: https://github.com/apache/spark/pull/28195#issuecomment-612649410 Merged build finished. Test PASSed. --

[GitHub] [spark] AmplabJenkins commented on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset URL: https://github.com/apache/spark/pull/28195#issuecomment-612649410 Merged build finished. Test PASSed. --

[GitHub] [spark] AmplabJenkins commented on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset URL: https://github.com/apache/spark/pull/28195#issuecomment-612649414 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28195: [SPARK-31425][SQL][CORE] UnsafeKVExternalSorter should also respect UnsafeAlignedOffset URL: https://github.com/apache/spark/pull/28195#issuecomment-612649414 Test PASSed. Refer to this link for build results (access rights to CI server ne

[GitHub] [spark] SparkQA commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check.

2020-04-12 Thread GitBox
SparkQA commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612649511 **[Test build #121145 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/121145/testR

[GitHub] [spark] SparkQA removed a comment on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check.

2020-04-12 Thread GitBox
SparkQA removed a comment on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612613510 **[Test build #121145 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12114

[GitHub] [spark] AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612649747 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.c

[GitHub] [spark] AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins commented on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612649745 Merged build finished. Test PASSed. This is

[GitHub] [spark] AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612649747 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://

[GitHub] [spark] AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check.

2020-04-12 Thread GitBox
AmplabJenkins removed a comment on issue #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#issuecomment-612649745 Merged build finished. Test PASSed.

[GitHub] [spark] srowen commented on a change in pull request #28182: [SPARK-29799][Streaming] add an implemention for issue SPARK-29799.

2020-04-12 Thread GitBox
srowen commented on a change in pull request #28182: [SPARK-29799][Streaming] add an implemention for issue SPARK-29799. URL: https://github.com/apache/spark/pull/28182#discussion_r407232547 ## File path: external/kafka-0-10/src/main/scala/org/apache/spark/streaming/kafka010/packag

[GitHub] [spark] srowen commented on a change in pull request #28182: [SPARK-29799][Streaming] add an implemention for issue SPARK-29799.

2020-04-12 Thread GitBox
srowen commented on a change in pull request #28182: [SPARK-29799][Streaming] add an implemention for issue SPARK-29799. URL: https://github.com/apache/spark/pull/28182#discussion_r407232507 ## File path: external/kafka-0-10/src/main/scala/org/apache/spark/streaming/kafka010/KafkaR

[GitHub] [spark] srowen commented on issue #28183: [SPARK-29791][CORE][SCHEDULER]add an implemention for issue SPARK-29791.

2020-04-12 Thread GitBox
srowen commented on issue #28183: [SPARK-29791][CORE][SCHEDULER]add an implemention for issue SPARK-29791. URL: https://github.com/apache/spark/pull/28183#issuecomment-612654550 I haven't looked at this in a while, but yeah this it better handled by the RM. In YARN you can over-provision c

[GitHub] [spark] gatorsmile commented on a change in pull request #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check.

2020-04-12 Thread GitBox
gatorsmile commented on a change in pull request #28194: [SPARK-31372][SQL][TEST] Display expression schema for double check. URL: https://github.com/apache/spark/pull/28194#discussion_r407235388 ## File path: sql/core/src/test/resources/sql-functions/output.out ## @@ -0,0

[GitHub] [spark] SparkQA commented on issue #28177: [SPARK-31407][SQL][TEST] Fix hive/SQLQuerySuite.derived from Hive query file: drop_database_removes_partition_dirs.q

2020-04-12 Thread GitBox
SparkQA commented on issue #28177: [SPARK-31407][SQL][TEST] Fix hive/SQLQuerySuite.derived from Hive query file: drop_database_removes_partition_dirs.q URL: https://github.com/apache/spark/pull/28177#issuecomment-612659364 **[Test build #121149 has finished](https://amplab.cs.berkeley.edu

  1   2   3   4   5   >