Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19687
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/19687#discussion_r149802015
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoderSuite.scala
---
@@ -441,4 +443,28 @@ class ExpressionEncoderS
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/19687#discussion_r149765467
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoderSuite.scala
---
@@ -370,7 +372,7 @@ class ExpressionEncoderSu
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19687#discussion_r149632280
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoderSuite.scala
---
@@ -370,7 +372,7 @@ class ExpressionEncoder
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19687#discussion_r149630182
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoderSuite.scala
---
@@ -441,4 +443,28 @@ class ExpressionEncoderSui
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19687#discussion_r149627183
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/encoders/ExpressionEncoderSuite.scala
---
@@ -441,4 +443,28 @@ class ExpressionEncoderSu
GitHub user zsxwing opened a pull request:
https://github.com/apache/spark/pull/19687
[SPARK-19644][SQL]Clean up Scala reflection garbage after creating Encoder
## What changes were proposed in this pull request?
Because of the memory leak issue in `scala.reflect.api.Types.T