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 ExpressionEncoderSuite extends PlanTest with
AnalysisTest {
private def encodeDecodeTest[T : ExpressionEncoder](
input: T,
testName: String): Unit = {
- test(s"encode/decode for $testName: $input") {
+ testAndVerifyNotLeakingReflectionObjects(s"encode/decode for
$testName: $input") {
val encoder = implicitly[ExpressionEncoder[T]]
--- End diff --
here we will verify the memory leak, seems no need to create
`testAndVerifyNotLeakingReflectionObjects`
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
