[GitHub] [spark] yaooqinn opened a new pull request #26626: [SPARK-29986][SQL] Introduce java like string trim to UTF8String

2019-11-21 Thread GitBox
yaooqinn opened a new pull request #26626: [SPARK-29986][SQL] Introduce java like string trim to UTF8String URL: https://github.com/apache/spark/pull/26626 ### What changes were proposed in this pull request? A java like string trim method trims all whitespaces that less

[GitHub] [spark] AmplabJenkins removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557032319 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557032310 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557032310 Merged build finished. Test PASSed. This is

[GitHub] [spark] AmplabJenkins commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557032319 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557031270 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
SparkQA commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557031858 **[Test build #114233 has

[GitHub] [spark] SparkQA removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
SparkQA removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557028658 **[Test build #114232 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557031259 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557031270 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557031259 Merged build finished. Test FAILed. This is

[GitHub] [spark] SparkQA commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
SparkQA commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557031237 **[Test build #114232 has

[GitHub] [spark] cloud-fan commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
cloud-fan commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557031062 cc @JkSelf @maryannxue This is an automated

[GitHub] [spark] normanmaurer commented on issue #26609: [SPARK-29971] Fix multiple possible buffer leaks in `TransportFrameDecoder/TransportCipher`

2019-11-21 Thread GitBox
normanmaurer commented on issue #26609: [SPARK-29971] Fix multiple possible buffer leaks in `TransportFrameDecoder/TransportCipher` URL: https://github.com/apache/spark/pull/26609#issuecomment-557030152 @vanzin like mentioned above I am not sure it is safe to assume it always consume all

[GitHub] [spark] cloud-fan commented on a change in pull request #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
cloud-fan commented on a change in pull request #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#discussion_r349015970 ## File path: sql/core/src/test/resources/sql-tests/inputs/comparator.sql ## @@ -1,3 +1,12 @@ --

[GitHub] [spark] maropu commented on a change in pull request #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
maropu commented on a change in pull request #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#discussion_r349016116 ## File path:

[GitHub] [spark] AmplabJenkins removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557029252 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557029266 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557029266 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] cloud-fan commented on a change in pull request #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
cloud-fan commented on a change in pull request #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#discussion_r349015186 ## File path: common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ## @@

[GitHub] [spark] maropu commented on a change in pull request #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
maropu commented on a change in pull request #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#discussion_r349015601 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557029252 Merged build finished. Test PASSed. This is

[GitHub] [spark] SparkQA commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
SparkQA commented on issue #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625#issuecomment-557028658 **[Test build #114232 has

[GitHub] [spark] cloud-fan commented on a change in pull request #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
cloud-fan commented on a change in pull request #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#discussion_r349013868 ## File path: common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ## @@

[GitHub] [spark] cloud-fan closed pull request #26500: [SPARK-29874][SQL]Optimize Dataset.isEmpty()

2019-11-21 Thread GitBox
cloud-fan closed pull request #26500: [SPARK-29874][SQL]Optimize Dataset.isEmpty() URL: https://github.com/apache/spark/pull/26500 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [spark] cloud-fan commented on issue #26500: [SPARK-29874][SQL]Optimize Dataset.isEmpty()

2019-11-21 Thread GitBox
cloud-fan commented on issue #26500: [SPARK-29874][SQL]Optimize Dataset.isEmpty() URL: https://github.com/apache/spark/pull/26500#issuecomment-557027063 thanks, merging to master! This is an automated message from the Apache

[GitHub] [spark] cloud-fan opened a new pull request #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader

2019-11-21 Thread GitBox
cloud-fan opened a new pull request #26625: [SPARK-29893][SQL][followup] code cleanup for local shuffle reader URL: https://github.com/apache/spark/pull/26625 ### What changes were proposed in this pull request? A few cleanups: 1. move the calculating of partition start

[GitHub] [spark] cloud-fan commented on a change in pull request #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
cloud-fan commented on a change in pull request #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#discussion_r349009278 ## File path:

[GitHub] [spark] cloud-fan commented on a change in pull request #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
cloud-fan commented on a change in pull request #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#discussion_r349008723 ## File path:

[GitHub] [spark] zhengruifeng commented on issue #26597: [SPARK-29960][ML][PYSPARK] MulticlassClassificationEvaluator support hammingLoss

2019-11-21 Thread GitBox
zhengruifeng commented on issue #26597: [SPARK-29960][ML][PYSPARK] MulticlassClassificationEvaluator support hammingLoss URL: https://github.com/apache/spark/pull/26597#issuecomment-557023455 Merged to master, thanks @huaxingao for reviewing!

[GitHub] [spark] zhengruifeng closed pull request #26597: [SPARK-29960][ML][PYSPARK] MulticlassClassificationEvaluator support hammingLoss

2019-11-21 Thread GitBox
zhengruifeng closed pull request #26597: [SPARK-29960][ML][PYSPARK] MulticlassClassificationEvaluator support hammingLoss URL: https://github.com/apache/spark/pull/26597 This is an automated message from the Apache Git

[GitHub] [spark] AmplabJenkins removed a comment on issue #21668: [SPARK-24690][SQL] Add a new config to control plan stats computation in LogicalRelation

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #21668: [SPARK-24690][SQL] Add a new config to control plan stats computation in LogicalRelation URL: https://github.com/apache/spark/pull/21668#issuecomment-557020065 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins removed a comment on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan URL: https://github.com/apache/spark/pull/26615#issuecomment-557019985 Test PASSed. Refer to this link for build results (access rights

[GitHub] [spark] AmplabJenkins removed a comment on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions URL: https://github.com/apache/spark/pull/25827#issuecomment-557020016 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins removed a comment on issue #21668: [SPARK-24690][SQL] Add a new config to control plan stats computation in LogicalRelation

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #21668: [SPARK-24690][SQL] Add a new config to control plan stats computation in LogicalRelation URL: https://github.com/apache/spark/pull/21668#issuecomment-557020057 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions URL: https://github.com/apache/spark/pull/25827#issuecomment-557020027 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan URL: https://github.com/apache/spark/pull/26615#issuecomment-557019971 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions URL: https://github.com/apache/spark/pull/25827#issuecomment-557020027 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #21668: [SPARK-24690][SQL] Add a new config to control plan stats computation in LogicalRelation

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #21668: [SPARK-24690][SQL] Add a new config to control plan stats computation in LogicalRelation URL: https://github.com/apache/spark/pull/21668#issuecomment-557020065 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #21668: [SPARK-24690][SQL] Add a new config to control plan stats computation in LogicalRelation

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #21668: [SPARK-24690][SQL] Add a new config to control plan stats computation in LogicalRelation URL: https://github.com/apache/spark/pull/21668#issuecomment-557020057 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions URL: https://github.com/apache/spark/pull/25827#issuecomment-557020016 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan URL: https://github.com/apache/spark/pull/26615#issuecomment-557019985 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins commented on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan URL: https://github.com/apache/spark/pull/26615#issuecomment-557019971 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA commented on issue #21668: [SPARK-24690][SQL] Add a new config to control plan stats computation in LogicalRelation

2019-11-21 Thread GitBox
SparkQA commented on issue #21668: [SPARK-24690][SQL] Add a new config to control plan stats computation in LogicalRelation URL: https://github.com/apache/spark/pull/21668#issuecomment-557019456 **[Test build #114231 has

[GitHub] [spark] SparkQA commented on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan

2019-11-21 Thread GitBox
SparkQA commented on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan URL: https://github.com/apache/spark/pull/26615#issuecomment-557019361 **[Test build #114229 has

[GitHub] [spark] SparkQA commented on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions

2019-11-21 Thread GitBox
SparkQA commented on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions URL: https://github.com/apache/spark/pull/25827#issuecomment-557019329 **[Test build #114230 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114230/testReport)**

[GitHub] [spark] zhengruifeng commented on issue #26575: [SPARK-29942][ML] Impl Complement Naive Bayes Classifier

2019-11-21 Thread GitBox
zhengruifeng commented on issue #26575: [SPARK-29942][ML] Impl Complement Naive Bayes Classifier URL: https://github.com/apache/spark/pull/26575#issuecomment-557019272 Merged to master, thanks @srowen for reviewing! This is

[GitHub] [spark] zhengruifeng closed pull request #26575: [SPARK-29942][ML] Impl Complement Naive Bayes Classifier

2019-11-21 Thread GitBox
zhengruifeng closed pull request #26575: [SPARK-29942][ML] Impl Complement Naive Bayes Classifier URL: https://github.com/apache/spark/pull/26575 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [spark] AmplabJenkins removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-557016595 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-557016604 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-557016595 Merged build finished. Test PASSed.

[GitHub] [spark] maropu commented on issue #21668: [SPARK-24690][SQL] Add a new config to control plan stats computation in LogicalRelation

2019-11-21 Thread GitBox
maropu commented on issue #21668: [SPARK-24690][SQL] Add a new config to control plan stats computation in LogicalRelation URL: https://github.com/apache/spark/pull/21668#issuecomment-557016748 retest this please This is an

[GitHub] [spark] AmplabJenkins commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-557016604 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] maropu commented on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions

2019-11-21 Thread GitBox
maropu commented on issue #25827: [SPARK-29128][SQL] Split predicate code in OR expressions URL: https://github.com/apache/spark/pull/25827#issuecomment-557016570 retest this please This is an automated message from the

[GitHub] [spark] maropu commented on a change in pull request #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan

2019-11-21 Thread GitBox
maropu commented on a change in pull request #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan URL: https://github.com/apache/spark/pull/26615#discussion_r348999209 ## File path:

[GitHub] [spark] SparkQA commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
SparkQA commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-557015944 **[Test build #114228 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-557014785 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
SparkQA removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-556970598 **[Test build #114217 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114217/testReport)** for

[GitHub] [spark] AmplabJenkins removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-557014778 Merged build finished. Test FAILed. This is an automated

[GitHub] [spark] AmplabJenkins commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-557014785 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-557014778 Merged build finished. Test FAILed. This is an automated message

[GitHub] [spark] SparkQA commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
SparkQA commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-557014406 **[Test build #114217 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114217/testReport)** for PR

[GitHub] [spark] AmplabJenkins removed a comment on issue #26624: [SPARK-8981][core] Add MDC support in Executor

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26624: [SPARK-8981][core] Add MDC support in Executor URL: https://github.com/apache/spark/pull/26624#issuecomment-557012699 Can one of the admins verify this patch? This is an

[GitHub] [spark] AmplabJenkins commented on issue #26624: [SPARK-8981][core] Add MDC support in Executor

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26624: [SPARK-8981][core] Add MDC support in Executor URL: https://github.com/apache/spark/pull/26624#issuecomment-557013179 Can one of the admins verify this patch? This is an automated

[GitHub] [spark] AmplabJenkins commented on issue #26624: [SPARK-8981][core] Add MDC support in Executor

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26624: [SPARK-8981][core] Add MDC support in Executor URL: https://github.com/apache/spark/pull/26624#issuecomment-557012699 Can one of the admins verify this patch? This is an automated

[GitHub] [spark] igreenfield opened a new pull request #26624: [SPARK-8981][core] Add MDC support in Executor

2019-11-21 Thread GitBox
igreenfield opened a new pull request #26624: [SPARK-8981][core] Add MDC support in Executor URL: https://github.com/apache/spark/pull/26624 ### What changes were proposed in this pull request? Added MDC support in all thread pulls TheardUtils create new pools that pass over MDC.

[GitHub] [spark] AmplabJenkins commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-557006822 Merged build finished. Test PASSed. This is an automated message

[GitHub] [spark] AmplabJenkins removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-557006822 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-557006834 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-557006834 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
SparkQA commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-557006276 **[Test build #114227 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114227/testReport)** for PR 26622

[GitHub] [spark] AmplabJenkins removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-557003539 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-557003527 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
SparkQA commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-557003382 **[Test build #114215 has

[GitHub] [spark] SparkQA removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
SparkQA removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-556967832 **[Test build #114215 has

[GitHub] [spark] AmplabJenkins commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-557003527 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-557003539 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] yaooqinn commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
yaooqinn commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-556998063 @cloud-fan if this works, we may need to fix Decimal separately This

[GitHub] [spark] AmplabJenkins removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-556997065 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-556997075 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-556997075 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-556997065 Merged build finished. Test PASSed. This is an automated message

[GitHub] [spark] SparkQA commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
SparkQA commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-556996428 **[Test build #114226 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114226/testReport)** for PR 26622

[GitHub] [spark] yaooqinn commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
yaooqinn commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-556994702 normal trim v.s. inside trim in toInt and toLong ```java [info] Java HotSpot(TM) 64-Bit Server VM 1.8.0_231-b11 on Mac OS X

[GitHub] [spark] AmplabJenkins removed a comment on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan URL: https://github.com/apache/spark/pull/26615#issuecomment-556990863 Test FAILed. Refer to this link for build results (access rights

[GitHub] [spark] SparkQA commented on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan

2019-11-21 Thread GitBox
SparkQA commented on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan URL: https://github.com/apache/spark/pull/26615#issuecomment-556990814 **[Test build #114224 has

[GitHub] [spark] AmplabJenkins commented on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan URL: https://github.com/apache/spark/pull/26615#issuecomment-556990863 Test FAILed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins removed a comment on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan URL: https://github.com/apache/spark/pull/26615#issuecomment-556990851 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA removed a comment on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan

2019-11-21 Thread GitBox
SparkQA removed a comment on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan URL: https://github.com/apache/spark/pull/26615#issuecomment-556981079 **[Test build #114224 has

[GitHub] [spark] AmplabJenkins commented on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26615: [SPARK-29977][SQL] Remove newMutableProjection/newOrdering/newNaturalAscendingOrdering from SparkPlan URL: https://github.com/apache/spark/pull/26615#issuecomment-556990851 Merged build finished. Test FAILed.

[GitHub] [spark] yaooqinn commented on a change in pull request #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
yaooqinn commented on a change in pull request #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#discussion_r348964345 ## File path: common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ## @@

[GitHub] [spark] AmplabJenkins removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-556986383 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
SparkQA removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-556978284 **[Test build #114223 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
AmplabJenkins removed a comment on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-556986377 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
SparkQA commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-556986882 **[Test build #114225 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114225/testReport)** for PR 26622

[GitHub] [spark] yaooqinn commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
yaooqinn commented on issue #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#issuecomment-556986541 normal trim v.s . no trim ``` [info] Java HotSpot(TM) 64-Bit Server VM 1.8.0_231-b11 on Mac OS X 10.15.1 [info] Intel(R)

[GitHub] [spark] AmplabJenkins commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-556986383 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
AmplabJenkins commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-556986377 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause

2019-11-21 Thread GitBox
SparkQA commented on issue #26623: [SPARK-29983][SQL] Add an independent config for an optional INTERVAL clause URL: https://github.com/apache/spark/pull/26623#issuecomment-556986342 **[Test build #114223 has

[GitHub] [spark] yaooqinn commented on a change in pull request #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim

2019-11-21 Thread GitBox
yaooqinn commented on a change in pull request #26622: [WIP][SPARK-28023][Test] Cheap UTF8String Trim URL: https://github.com/apache/spark/pull/26622#discussion_r348961970 ## File path: common/unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java ## @@

<    4   5   6   7   8   9   10   11   >