Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22637
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user Fokko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22637#discussion_r223196616
--- Diff:
sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/CLIService.java
---
@@ -154,6 +154,7 @@ public synchronized void start() {
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22637#discussion_r223188294
--- Diff:
sql/catalyst/src/test/java/org/apache/spark/sql/catalyst/expressions/RowBasedKeyValueBatchSuite.java
---
@@ -152,31 +151,24 @@ public void
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22637#discussion_r223188308
--- Diff:
sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/CLIService.java
---
@@ -154,6 +154,7 @@ public synchronized void start() {
Github user Fokko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22637#discussion_r223187949
--- Diff:
sql/catalyst/src/test/java/org/apache/spark/sql/catalyst/expressions/RowBasedKeyValueBatchSuite.java
---
@@ -152,31 +151,27 @@ public void
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22637#discussion_r223187058
--- Diff:
sql/catalyst/src/test/java/org/apache/spark/sql/catalyst/expressions/RowBasedKeyValueBatchSuite.java
---
@@ -152,31 +151,27 @@ public void
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22637#discussion_r223102153
--- Diff:
common/network-shuffle/src/test/java/org/apache/spark/network/shuffle/ExternalShuffleSecuritySuite.java
---
@@ -96,14 +96,16 @@ private
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22637#discussion_r223101514
--- Diff:
common/network-shuffle/src/test/java/org/apache/spark/network/shuffle/ExternalShuffleBlockResolverSuite.java
---
@@ -98,19 +98,19 @@
Github user Fokko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22637#discussion_r222893178
--- Diff:
common/network-shuffle/src/test/java/org/apache/spark/network/shuffle/ExternalShuffleIntegrationSuite.java
---
@@ -133,37 +133,38 @@ private
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22637#discussion_r222892548
--- Diff:
common/network-shuffle/src/test/java/org/apache/spark/network/shuffle/ExternalShuffleIntegrationSuite.java
---
@@ -133,37 +133,38 @@ private
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22637#discussion_r222892759
--- Diff:
sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/CLIService.java
---
@@ -154,6 +154,7 @@ public synchronized void start() {
GitHub user Fokko opened a pull request:
https://github.com/apache/spark/pull/22637
Spark 25408
## What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
## How was this patch tested?
(Please explain how this patch
12 matches
Mail list logo
