Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/19718
Thanks! Merging to 2.2.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user zsxwing closed the pull request at:
https://github.com/apache/spark/pull/19718
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19630
**[Test build #83701 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83701/testReport)**
for PR 19630 at commit
[`75d31b4`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19630
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83701/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19630
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19719
**[Test build #83708 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83708/testReport)**
for PR 19719 at commit
[`0ce241d`](https://github.com/apache/spark/commit/0c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19630
**[Test build #83709 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83709/testReport)**
for PR 19630 at commit
[`94cded6`](https://github.com/apache/spark/commit/94
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19433#discussion_r150352747
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/tree/model/InformationGainStats.scala
---
@@ -112,7 +113,7 @@ private[spark] object ImpurityStats {
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19433#discussion_r150154413
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/AggUpdateUtils.scala ---
@@ -0,0 +1,86 @@
+/*
+ * Licensed to the Apache Software Fou
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19433#discussion_r150309552
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/SplitUtils.scala ---
@@ -0,0 +1,215 @@
+/*
+ * Licensed to the Apache Software Founda
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19433#discussion_r150160368
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/SplitUtils.scala ---
@@ -0,0 +1,215 @@
+/*
+ * Licensed to the Apache Software Founda
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19433#discussion_r150159113
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/SplitUtils.scala ---
@@ -0,0 +1,215 @@
+/*
+ * Licensed to the Apache Software Founda
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19433#discussion_r150159513
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/SplitUtils.scala ---
@@ -0,0 +1,215 @@
+/*
+ * Licensed to the Apache Software Founda
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19433#discussion_r150158027
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/RandomForest.scala ---
@@ -627,221 +621,37 @@ private[spark] object RandomForest extends Logg
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19433#discussion_r150349566
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/RandomForest.scala ---
@@ -852,6 +662,41 @@ private[spark] object RandomForest extends Loggin
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19720
**[Test build #83707 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83707/testReport)**
for PR 19720 at commit
[`f0edd7e`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19720
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83707/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19720
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19719
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19719
**[Test build #83708 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83708/testReport)**
for PR 19719 at commit
[`0ce241d`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19719
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83708/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19719
**[Test build #83705 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83705/testReport)**
for PR 19719 at commit
[`7b767bf`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19719
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19719
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83705/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19459
**[Test build #83703 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83703/testReport)**
for PR 19459 at commit
[`6c72e37`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19459
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19459
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83703/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19407
**[Test build #83704 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83704/testReport)**
for PR 19407 at commit
[`788fbf3`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19407
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83704/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19407
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/19407
Thanks! Merging to master and 2.2.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19719#discussion_r150358375
--- Diff:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala
---
@@ -521,7 +521,20 @@ class HiveT
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19407
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19077
According to [this
material](http://s3-eu-west-1.amazonaws.com/presentations2013/10_presentation.pdf#pages=42),
`Unsafe.allocateMemory` uses 8-byte boundary.
---
---
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19601
@cloud-fan could you please review this again? Now, this PR does not apply
any change to `ColumnVector` and `WritableColumnVector`.
---
--
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r150361985
--- Diff: mllib/src/main/scala/org/apache/spark/ml/image/ImageSchema.scala
---
@@ -0,0 +1,236 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19719#discussion_r150362312
--- Diff:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala
---
@@ -521,7 +521,20 @@ class HiveTh
Github user sergiobilello-eb commented on the issue:
https://github.com/apache/spark/pull/18134
This function is not reported in the API documentation:
https://spark.apache.org/docs/2.2.0/api/java/index.html?org/apache/spark/sql/functions.html
or
https://spark.apache.org/docs/2.1.0/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19651
**[Test build #83710 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83710/testReport)**
for PR 19651 at commit
[`4b09faa`](https://github.com/apache/spark/commit/4b
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18134
@sergiobilello-eb This is just a SQL function. You can call it in SQL
interface or using the `df.select(expr("dayofweek('2009-07-30')"))`. It is not
part of the DataFrame functions. You can submi
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/19719#discussion_r150363309
--- Diff:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala
---
@@ -521,7 +521,20 @@ class HiveT
Github user sergiobilello-eb commented on the issue:
https://github.com/apache/spark/pull/18134
thanks @gatorsmile :)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mai
Github user sergiobilello-eb commented on the issue:
https://github.com/apache/spark/pull/18134
```spark-sql> select dayofweek('2007-02-03'), dayofweek('2009-07-30'),
dayofweek('2017-05-27'), dayofweek(null), dayofweek('1582-10-15 13:10:15');
17/11/10 16:08:23 INFO SparkSqlParser:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19689
**[Test build #83711 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83711/testReport)**
for PR 19689 at commit
[`20dc217`](https://github.com/apache/spark/commit/20
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18134
https://issues.apache.org/jira/browse/SPARK-20909
This is not part of 2.2. Based on JIRA, it will be included in 2.3
---
--
Github user sergiobilello-eb commented on the issue:
https://github.com/apache/spark/pull/18134
thanks @gatorsmile! Do you suggest any workaround until then? I mean not
rebuilding spark with that patch Can I register my UDF that contains that
logic?
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18134
Yeah, you always can implement such a UDF.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user yaooqinn commented on a diff in the pull request:
https://github.com/apache/spark/pull/19719#discussion_r150364544
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveUtils.scala
---
@@ -66,6 +66,12 @@ private[spark] object HiveUtils extends Logging {
Github user yaooqinn commented on a diff in the pull request:
https://github.com/apache/spark/pull/19719#discussion_r150364666
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveUtils.scala
---
@@ -66,6 +66,12 @@ private[spark] object HiveUtils extends Logging {
Github user yaooqinn commented on the issue:
https://github.com/apache/spark/pull/19719
make it an AlternativeConfig maybe better
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comm
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/19640
@cloud-fan For the UI part, how about this PR:
https://github.com/apache/spark/pull/14577
---
-
To unsubscribe, e-mail: reviews-
Github user yaooqinn commented on a diff in the pull request:
https://github.com/apache/spark/pull/19719#discussion_r150366200
--- Diff:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala
---
@@ -521,7 +521,20 @@ class HiveThr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19719
**[Test build #83712 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83712/testReport)**
for PR 19719 at commit
[`28fab02`](https://github.com/apache/spark/commit/28
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19719#discussion_r150366512
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveUtils.scala
---
@@ -66,6 +66,12 @@ private[spark] object HiveUtils extends Logging {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19719#discussion_r150366571
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveUtils.scala
---
@@ -66,6 +66,12 @@ private[spark] object HiveUtils extends Logging {
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19640
I think it makes sense, but the implementation is too hacky. We should
extract the time epoch and convert it to timestamp string with local timezone.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r150367100
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcDeserializer.scala
---
@@ -0,0 +1,206 @@
+/*
+ * Licensed to
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r150367190
--- Diff: mllib/src/main/scala/org/apache/spark/ml/image/HadoopUtils.scala
---
@@ -0,0 +1,109 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r150367382
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcDeserializer.scala
---
@@ -0,0 +1,206 @@
+/*
+ * Licensed to
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r150367580
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcFilters.scala
---
@@ -0,0 +1,183 @@
+/*
+ * Licensed to the A
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r150367867
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcSerializer.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to th
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r150367926
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcUtils.scala
---
@@ -0,0 +1,131 @@
+/*
+ * Licensed to the Apa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19719
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83712/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19719
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19719
**[Test build #83712 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83712/testReport)**
for PR 19719 at commit
[`28fab02`](https://github.com/apache/spark/commit/2
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/19439
Thanks for the updates! My only remaining comments are about:
* Default arguments for readImages in Scala not being Java-friendly (I'd
still recommend taking the easy route by having 1 simple
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r150368108
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcSerializer.scala
---
@@ -0,0 +1,163 @@
+/*
+ * Licensed to th
Github user yaooqinn commented on a diff in the pull request:
https://github.com/apache/spark/pull/19719#discussion_r150368111
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveUtils.scala
---
@@ -66,6 +66,12 @@ private[spark] object HiveUtils extends Logging {
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/19714#discussion_r150368276
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -154,12 +158,12 @@ abstract class SparkStrategies extends
Que
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19719
**[Test build #83713 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83713/testReport)**
for PR 19719 at commit
[`42d05a7`](https://github.com/apache/spark/commit/42
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19630
**[Test build #83709 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83709/testReport)**
for PR 19630 at commit
[`94cded6`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19630
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83709/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19630
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user yaooqinn commented on a diff in the pull request:
https://github.com/apache/spark/pull/19719#discussion_r150368620
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLEnv.scala
---
@@ -55,6 +55,7 @@ private[hive] object SparkSQ
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19714
**[Test build #83714 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83714/testReport)**
for PR 19714 at commit
[`c78951b`](https://github.com/apache/spark/commit/c7
Github user yaooqinn commented on a diff in the pull request:
https://github.com/apache/spark/pull/19719#discussion_r150368744
--- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveUtils.scala
---
@@ -66,6 +66,12 @@ private[spark] object HiveUtils extends Logging {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19719
**[Test build #83713 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83713/testReport)**
for PR 19719 at commit
[`42d05a7`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19719
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83713/
Test FAILed.
---
Github user yaooqinn commented on a diff in the pull request:
https://github.com/apache/spark/pull/19719#discussion_r150368818
--- Diff:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala
---
@@ -521,7 +521,20 @@ class HiveThr
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19719
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19651#discussion_r150369899
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcFilters.scala
---
@@ -0,0 +1,183 @@
+/*
+ * Licensed to t
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19651
**[Test build #83710 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83710/testReport)**
for PR 19651 at commit
[`4b09faa`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19651
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19651
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83710/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19689
**[Test build #83711 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83711/testReport)**
for PR 19689 at commit
[`20dc217`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19689
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19689
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83711/
Test PASSed.
---
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/19208#discussion_r150373516
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/CrossValidator.scala ---
@@ -117,6 +123,12 @@ class CrossValidator @Since("1.2.0") (@Since("1.4.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19714
**[Test build #83714 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83714/testReport)**
for PR 19714 at commit
[`c78951b`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19714
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19714
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83714/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19077
**[Test build #83715 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83715/testReport)**
for PR 19077 at commit
[`967c73a`](https://github.com/apache/spark/commit/96
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19690
**[Test build #83716 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83716/testReport)**
for PR 19690 at commit
[`37fba60`](https://github.com/apache/spark/commit/37
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r150375592
--- Diff: mllib/src/main/scala/org/apache/spark/ml/image/ImageSchema.scala
---
@@ -0,0 +1,239 @@
+/*
+ * Licensed to the Apache Software Foundat
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r150375543
--- Diff: mllib/src/main/scala/org/apache/spark/ml/image/ImageSchema.scala
---
@@ -0,0 +1,239 @@
+/*
+ * Licensed to the Apache Software Foundat
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r150375535
--- Diff: mllib/src/main/scala/org/apache/spark/ml/image/ImageSchema.scala
---
@@ -0,0 +1,239 @@
+/*
+ * Licensed to the Apache Software Foundat
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r150375473
--- Diff: python/pyspark/ml/image.py ---
@@ -0,0 +1,196 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contri
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19713
**[Test build #83717 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83717/testReport)**
for PR 19713 at commit
[`c491974`](https://github.com/apache/spark/commit/c4
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r150376396
--- Diff: python/pyspark/ml/image.py ---
@@ -0,0 +1,196 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contri
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/19543#discussion_r150376571
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/deploy/mesos/config.scala
---
@@ -122,4 +122,11 @@ package object config {
301 - 400 of 411 matches
Mail list logo