Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14836
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is ena
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14836#discussion_r76646997
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala
---
@@ -1452,7 +1452,6 @@ class DataFrameSuite extends QueryTest with
SharedSQ
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14836#discussion_r76575766
--- Diff: core/src/test/scala/org/apache/spark/AccumulatorSuite.scala ---
@@ -171,7 +173,7 @@ class AccumulatorSuite extends SparkFunSuite with
Matchers with
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14836#discussion_r76575699
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala
---
@@ -1452,7 +1452,6 @@ class DataFrameSuite extends QueryTest with
SharedSQLCo
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14836#discussion_r76502969
--- Diff: core/src/test/scala/org/apache/spark/AccumulatorSuite.scala ---
@@ -171,7 +171,7 @@ class AccumulatorSuite extends SparkFunSuite with
Matchers wi
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14836#discussion_r76502012
--- Diff: core/src/test/scala/org/apache/spark/AccumulatorSuite.scala ---
@@ -100,7 +100,7 @@ class AccumulatorSuite extends SparkFunSuite with
Matchers wi
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14836#discussion_r76499643
--- Diff: core/src/test/scala/org/apache/spark/AccumulatorSuite.scala ---
@@ -171,7 +171,7 @@ class AccumulatorSuite extends SparkFunSuite with
Matchers with Lo
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14836#discussion_r76499489
--- Diff: core/src/test/scala/org/apache/spark/AccumulatorSuite.scala ---
@@ -100,7 +100,7 @@ class AccumulatorSuite extends SparkFunSuite with
Matchers with Lo
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14836#discussion_r76491325
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1238,7 +1238,7 @@ class SparkContext(config: SparkConf) extends Logging
with Exec
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14836#discussion_r76488765
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1238,7 +1238,7 @@ class SparkContext(config: SparkConf) extends Logging
with Executo
GitHub user keypointt opened a pull request:
https://github.com/apache/spark/pull/14836
[MINOR][MLlib][SQL] Clean up unused variables and unused import
## What changes were proposed in this pull request?
Clean up unused variables and unused import statements, unnecessary
`r
11 matches
Mail list logo