-
- Downloads
[SPARK-18387][SQL] Add serialization to checkEvaluation.
## What changes were proposed in this pull request? This removes the serialization test from RegexpExpressionsSuite and replaces it by serializing all expressions in checkEvaluation. This also fixes math constant expressions by making LeafMathExpression Serializable and fixes NumberFormat values that are null or invalid after serialization. ## How was this patch tested? This patch is to tests. Author: Ryan Blue <blue@apache.org> Closes #15847 from rdblue/SPARK-18387-fix-serializable-expressions. (cherry picked from commit 6e95325f) Signed-off-by:Reynold Xin <rxin@databricks.com>
Showing
- sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala 1 addition, 1 deletion...ache/spark/sql/catalyst/expressions/mathExpressions.scala
- sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala 25 additions, 19 deletions...he/spark/sql/catalyst/expressions/stringExpressions.scala
- sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ExpressionEvalHelper.scala 9 additions, 6 deletions...spark/sql/catalyst/expressions/ExpressionEvalHelper.scala
- sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/RegexpExpressionsSuite.scala 1 addition, 15 deletions...ark/sql/catalyst/expressions/RegexpExpressionsSuite.scala
Loading
Please register or sign in to comment