Github user rberenguel commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146177319
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -270,12 +271,25 @@ trait CheckAnalysis extends P
Github user rberenguel commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146177284
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -24,6 +24,7 @@ import
org.apache.spark.sql.cata
Github user rberenguel commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146177263
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -270,12 +271,25 @@ trait CheckAnalysis extends P
Github user rberenguel commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146177196
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -270,12 +271,25 @@ trait CheckAnalysis extends P
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146177053
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -270,12 +271,25 @@ trait CheckAnalysis extends Predi
Github user rberenguel commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146176760
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -270,12 +271,25 @@ trait CheckAnalysis extends P
Github user rberenguel commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146176626
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -24,6 +24,7 @@ import
org.apache.spark.sql.cata
Github user rberenguel commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146176609
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -270,12 +271,25 @@ trait CheckAnalysis extends P
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19548
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/19548
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82968/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19548
**[Test build #82968 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82968/testReport)**
for PR 19548 at commit
[`28c7ce8`](https://github.com/apache/spark/commit/2
Github user sjrand closed the pull request at:
https://github.com/apache/spark/pull/19554
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user sjrand commented on the issue:
https://github.com/apache/spark/pull/19554
Thanks @jerryshao.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h..
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19554
@sjrand would you please close this PR, it is already merged to branch 2.2.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@s
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19554
Thanks, merging to branch 2.2.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-ma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19554
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/19554
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82967/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19554
**[Test build #82967 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82967/testReport)**
for PR 19554 at commit
[`def29a1`](https://github.com/apache/spark/commit/d
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17862#discussion_r146169405
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala ---
@@ -42,7 +44,26 @@ import org.apache.spark.sql.functions.{col, lit}
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17862#discussion_r146168734
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala ---
@@ -282,8 +348,27 @@ class LinearSVC @Since("2.2.0") (
@Since("2.2
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17862#discussion_r146168660
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala ---
@@ -42,7 +44,26 @@ import org.apache.spark.sql.functions.{col, lit}
Github user hhbyyh commented on the issue:
https://github.com/apache/spark/pull/19439
@thunterdb Thanks for the reply.
> It does, indirectly: this is what the field types CV_32FXX do. You need
to do some low-level casting to convert the byte array to array of numbers, but
th
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17862#discussion_r146167919
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala ---
@@ -282,8 +348,27 @@ class LinearSVC @Since("2.2.0") (
@Sinc
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17862#discussion_r146167706
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala ---
@@ -42,7 +44,26 @@ import org.apache.spark.sql.functions.{col, li
Github user hhbyyh commented on the issue:
https://github.com/apache/spark/pull/17862
Thanks @WeichenXu123 for the comments.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17862#discussion_r146166006
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala ---
@@ -282,8 +348,27 @@ class LinearSVC @Since("2.2.0") (
@Since("2.2
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17862#discussion_r146165706
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala ---
@@ -282,8 +348,27 @@ class LinearSVC @Since("2.2.0") (
@Since("2.2
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17862#discussion_r146165449
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LinearSVC.scala ---
@@ -42,7 +44,26 @@ import org.apache.spark.sql.functions.{col, lit}
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r146163447
--- Diff: mllib/src/main/scala/org/apache/spark/ml/image/ImageSchema.scala
---
@@ -0,0 +1,258 @@
+/*
+ * Licensed to the Apache Software Founda
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r146163650
--- Diff: python/pyspark/ml/image.py ---
@@ -0,0 +1,122 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contr
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r146164215
--- Diff: python/pyspark/ml/image.py ---
@@ -0,0 +1,122 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contr
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19439#discussion_r146163724
--- Diff: python/pyspark/ml/image.py ---
@@ -0,0 +1,122 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contr
Github user nkronenfeld commented on the issue:
https://github.com/apache/spark/pull/19529
Documentation removed as per @srowen 's request in the associated JIRA
issue [SPARK-22308]
---
-
To unsubscribe, e-mail: re
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19529
**[Test build #82969 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82969/testReport)**
for PR 19529 at commit
[`4218b86`](https://github.com/apache/spark/commit/42
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19506
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82966/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19506
Merged build finished. Test PASSed.
---
-
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/19506
**[Test build #82966 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82966/testReport)**
for PR 19506 at commit
[`1e95a2f`](https://github.com/apache/spark/commit/1
Github user falaki commented on a diff in the pull request:
https://github.com/apache/spark/pull/19551#discussion_r146160321
--- Diff: R/pkg/tests/fulltests/test_sparkSQL.R ---
@@ -499,6 +499,12 @@ test_that("create DataFrame with different data
types", {
expect_equal(collec
Github user falaki commented on a diff in the pull request:
https://github.com/apache/spark/pull/19551#discussion_r146160421
--- Diff: R/pkg/R/DataFrame.R ---
@@ -1191,6 +1191,9 @@ setMethod("collect",
vec <- do.call(c, col)
stopifno
Github user taroplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/19548#discussion_r146160084
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/jdbc/JDBCSuite.scala
---
@@ -815,6 +815,12 @@ class JDBCSuite extends SparkFunSuite
Some(
Github user taroplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/19548#discussion_r146160039
--- Diff:
external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/OracleIntegrationSuite.scala
---
@@ -108,6 +112,10 @@ class OracleInt
Github user taroplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/19548#discussion_r146160051
--- Diff:
external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/OracleIntegrationSuite.scala
---
@@ -307,4 +315,32 @@ class OracleInt
Github user taroplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/19548#discussion_r146160013
--- Diff:
external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/OracleIntegrationSuite.scala
---
@@ -50,9 +52,11 @@ import org.apache
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19548
**[Test build #82968 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82968/testReport)**
for PR 19548 at commit
[`28c7ce8`](https://github.com/apache/spark/commit/28
Github user taroplus commented on a diff in the pull request:
https://github.com/apache/spark/pull/19548#discussion_r146159980
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/jdbc/OracleDialect.scala ---
@@ -28,25 +28,28 @@ private case object OracleDialect extends JdbcDiale
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19554
**[Test build #82967 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82967/testReport)**
for PR 19554 at commit
[`def29a1`](https://github.com/apache/spark/commit/de
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19554
ok to test.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19554
Can you please add a tag in PR title `[BACKPORT-2.2]`.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19554
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19554
ok to test.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
GitHub user sjrand opened a pull request:
https://github.com/apache/spark/pull/19554
[SPARK-22319][Core] call loginUserFromKeytab before accessing hdfs
## What changes were proposed in this pull request?
In SparkSubmit, call loginUserFromKeytab before attempting to make RPC
Github user sjrand commented on the issue:
https://github.com/apache/spark/pull/19540
Thanks @jerryshao and @jiangxb1987. Created
https://github.com/apache/spark/pull/19554 for branch-2.2.
---
-
To unsubscribe, e-ma
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18747#discussion_r146157000
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala
---
@@ -23,21 +23,70 @@ import org.apache.spark.sql.ca
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18747#discussion_r146156401
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala
---
@@ -23,21 +23,70 @@ import org.apache.spark.sql.ca
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18747#discussion_r146155764
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/columnar/InMemoryTableScanExec.scala
---
@@ -23,21 +23,70 @@ import org.apache.spark.sql.ca
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19534
@sitalkedia I'm OK with either.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/19519#discussion_r146154530
--- Diff:
core/src/main/scala/org/apache/spark/deploy/SparkApplication.scala ---
@@ -0,0 +1,55 @@
+/*
+ * Licensed to the Apache Software Foundati
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146153863
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -270,12 +271,25 @@ trait CheckAnalysis extends Predic
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146153187
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -270,12 +271,25 @@ trait CheckAnalysis extends Predic
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146153590
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -270,12 +271,25 @@ trait CheckAnalysis extends Predic
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146153288
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -270,12 +271,25 @@ trait CheckAnalysis extends Predic
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146153739
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -270,12 +271,25 @@ trait CheckAnalysis extends Predic
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146152572
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -24,6 +24,7 @@ import
org.apache.spark.sql.catalyst.
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146153805
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -270,12 +271,25 @@ trait CheckAnalysis extends Predi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19548
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19540
@sjrand , can you please create another PR against branch-2.2, it is not
auto-mergeable, thanks!
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19548
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82965/
Test PASSed.
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19540
---
-
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/19548
**[Test build #82965 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82965/testReport)**
for PR 19548 at commit
[`91e911d`](https://github.com/apache/spark/commit/9
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/19540
LGTM, merging to master and branch 2.2.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/19527
@BryanCutler @MLnick @WeichenXu123 Thanks for reviewing. Your comments
should be all addressed now. Please take a look again when you have more time.
---
---
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19527#discussion_r146153067
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/OneHotEncoderEstimator.scala
---
@@ -0,0 +1,464 @@
+/*
+ * Licensed to the Apache Softwar
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146152918
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -270,12 +271,25 @@ trait CheckAnalysis extends Predi
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146152895
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisErrorSuite.scala
---
@@ -408,16 +408,28 @@ class AnalysisErrorSuite ext
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17100#discussion_r146152740
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -24,6 +24,7 @@ import
org.apache.spark.sql.catalyst
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/19506#discussion_r146151411
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/ApproxCountDistinctForIntervals.scala
---
@@ -239,4 +219,26 @@ case
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19506
**[Test build #82966 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82966/testReport)**
for PR 19506 at commit
[`1e95a2f`](https://github.com/apache/spark/commit/1e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19451
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82964/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19451
Merged build finished. Test PASSed.
---
-
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/19451
**[Test build #82964 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82964/testReport)**
for PR 19451 at commit
[`7030ab6`](https://github.com/apache/spark/commit/7
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19553#discussion_r146147996
--- Diff: core/src/test/scala/org/apache/spark/util/JavaUtils.scala ---
@@ -0,0 +1,37 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) unde
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19553#discussion_r146147889
--- Diff: core/src/main/scala/org/apache/spark/api/java/JavaUtils.scala ---
@@ -43,6 +43,13 @@ private[spark] object JavaUtils {
override def s
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19553#discussion_r146147952
--- Diff: core/src/test/scala/org/apache/spark/util/JavaUtils.scala ---
@@ -0,0 +1,37 @@
+/*
--- End diff --
This class needs to be in the sa
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19548#discussion_r146147276
--- Diff:
external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/OracleIntegrationSuite.scala
---
@@ -108,6 +112,10 @@ class OracleInteg
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19548#discussion_r146147286
--- Diff:
external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/OracleIntegrationSuite.scala
---
@@ -50,9 +52,11 @@ import org.apache.s
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19548#discussion_r146147293
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/jdbc/JDBCSuite.scala
---
@@ -815,6 +815,12 @@ class JDBCSuite extends SparkFunSuite
Some(De
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19548#discussion_r146147267
--- Diff:
external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/OracleIntegrationSuite.scala
---
@@ -307,4 +315,32 @@ class OracleInteg
Github user taroplus commented on the issue:
https://github.com/apache/spark/pull/19548
Okay new integration test has been added
According to their
[document](https://docs.oracle.com/database/121/TTSQL/types.htm#TTSQL148)
> BINARY_DOUBLE is a 64-bit, double-precision
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19548
**[Test build #82965 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82965/testReport)**
for PR 19548 at commit
[`91e911d`](https://github.com/apache/spark/commit/91
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19451
**[Test build #82964 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82964/testReport)**
for PR 19451 at commit
[`7030ab6`](https://github.com/apache/spark/commit/70
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19553
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user Whoosh opened a pull request:
https://github.com/apache/spark/pull/19553
[SPARK-22330][CORE] Linear containsKey operation for serialized maps
â¦alization.
## What changes were proposed in this pull request?
Use non-linear containsKey operation for seria
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19451
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82962/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19451
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/19451
**[Test build #82962 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82962/testReport)**
for PR 19451 at commit
[`f6c95d3`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19552
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82961/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19552
Merged build finished. Test PASSed.
---
-
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/19552
**[Test build #82961 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82961/testReport)**
for PR 19552 at commit
[`a256627`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19543
**[Test build #82963 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82963/testReport)**
for PR 19543 at commit
[`c587946`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19543
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82963/
Test PASSed.
---
1 - 100 of 166 matches
Mail list logo
