Skip to content

Commit 7d44c01

Browse files
cloud-fanmarmbrus
authored andcommitted
[SQL] [MINOR] [THIS] use private for internal field in ScalaUdf
Author: Wenchen Fan <[email protected]> Closes #6235 from cloud-fan/tmp and squashes the following commits: 8f16367 [Wenchen Fan] use private[this] (cherry picked from commit 56ede88) Signed-off-by: Michael Armbrust <[email protected]>
1 parent c7623a2 commit 7d44c01

File tree

1 file changed

+4
-4
lines changed
  • sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions

1 file changed

+4
-4
lines changed

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUdf.scala

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,9 @@ case class ScalaUdf(function: AnyRef, dataType: DataType, children: Seq[Expressi
5555
}.foreach(println)
5656
5757
*/
58-
59-
val f = children.size match {
60-
case 0 =>
58+
59+
private[this] val f = children.size match {
60+
case 0 =>
6161
val func = function.asInstanceOf[() => Any]
6262
(input: Row) => {
6363
func()
@@ -956,7 +956,7 @@ case class ScalaUdf(function: AnyRef, dataType: DataType, children: Seq[Expressi
956956
}
957957

958958
// scalastyle:on
959-
val converter = CatalystTypeConverters.createToCatalystConverter(dataType)
959+
private[this] val converter = CatalystTypeConverters.createToCatalystConverter(dataType)
960960
override def eval(input: Row): Any = converter(f(input))
961961

962962
}

0 commit comments

Comments
 (0)