[GitHub] spark pull request #20825: add impurity stats in tree leaf node debug string

2018-05-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/20825 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request #20857: [SPARK-23735]Optimize the document by adding an i...

2018-05-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/20857 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request #20881: Add a note about jobs running in FIFO order in th...

2018-05-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/20881 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark issue #20869: Improve implicitNotFound message for Encoder

2018-05-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20869 **[Test build #4173 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4173/testReport)** for PR 20869 at commit

[GitHub] spark issue #21273: [SPARK-17916][SQL] Fix empty string being parsed as null...

2018-05-12 Thread gengliangwang
Github user gengliangwang commented on the issue: https://github.com/apache/spark/pull/21273 LGTM, it would be nice to have a micro-benmark suite in this PR. --- - To unsubscribe, e-mail:

[GitHub] spark pull request #21298: [SPARK-24198][SparkR][SQL] Adding slice function ...

2018-05-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/21298 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark issue #21311: [SPARK-24257][SQL]LongToUnsafeRowMap calculate the new s...

2018-05-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21311 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21311: [SPARK-24257][SQL]LongToUnsafeRowMap calculate the new s...

2018-05-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21311 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21298: [SPARK-24198][SparkR][SQL] Adding slice function to Spar...

2018-05-12 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/21298 Merged to master. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark pull request #21311: [SPARK-24257][SQL]LongToUnsafeRowMap calculate th...

2018-05-12 Thread cxzl25
GitHub user cxzl25 opened a pull request: https://github.com/apache/spark/pull/21311 [SPARK-24257][SQL]LongToUnsafeRowMap calculate the new size may be wrong ## What changes were proposed in this pull request? LongToUnsafeRowMap Calculate the new size simply by

[GitHub] spark pull request #21247: [SPARK-24190] Separating JSONOptions for read

2018-05-12 Thread gengliangwang
Github user gengliangwang commented on a diff in the pull request: https://github.com/apache/spark/pull/21247#discussion_r187769734 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala --- @@ -138,3 +121,40 @@ private[sql] class JSONOptions(

[GitHub] spark issue #21298: [SPARK-24198][SparkR][SQL] Adding slice function to Spar...

2018-05-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21298 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21298: [SPARK-24198][SparkR][SQL] Adding slice function to Spar...

2018-05-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21298 **[Test build #90539 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90539/testReport)** for PR 21298 at commit

[GitHub] spark issue #21298: [SPARK-24198][SparkR][SQL] Adding slice function to Spar...

2018-05-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21298 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90539/ Test PASSed. ---

[GitHub] spark pull request #21246: [SPARK-23901][SQL] Add masking functions

2018-05-12 Thread mgaido91
Github user mgaido91 commented on a diff in the pull request: https://github.com/apache/spark/pull/21246#discussion_r187769261 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/maskExpressions.scala --- @@ -0,0 +1,569 @@ +/* + * Licensed to

[GitHub] spark issue #21309: [SPARK-23907] Removes regr_* functions in functions.scal...

2018-05-12 Thread mgaido91
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/21309 @rxin I also always had some problems in understanding which functions are available and which not. I think it is quite confusing for a user. I am wondering: what do you think if we automatically

[GitHub] spark issue #21298: [SPARK-24198][SparkR][SQL] Adding slice function to Spar...

2018-05-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21298 **[Test build #90539 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90539/testReport)** for PR 21298 at commit

[GitHub] spark issue #20611: [SPARK-23425][SQL]Support wildcard in HDFS path for load...

2018-05-12 Thread sujith71955
Github user sujith71955 commented on the issue: https://github.com/apache/spark/pull/20611 @kevinyu98 Spark supports wildcards characters in both local and non local scenarios, the only restriction spark imposes is using wildcards in folder names of the local file-system path ,

[GitHub] spark issue #21310: [SPARK-24256][SQL] SPARK-24256: ExpressionEncoder should...

2018-05-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21310 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21310: [SPARK-24256][SQL] SPARK-24256: ExpressionEncoder should...

2018-05-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21310 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark pull request #21310: [SPARK-24256][SQL] SPARK-24256: ExpressionEncoder...

2018-05-12 Thread fangshil
GitHub user fangshil opened a pull request: https://github.com/apache/spark/pull/21310 [SPARK-24256][SQL] SPARK-24256: ExpressionEncoder should support user-defined types as fields of Scala case class and tuple ## What changes were proposed in this pull request? Right now,

<    1   2