Skip to content

Commit

Permalink
[SQL] [MINOR] [THIS] use private for internal field in ScalaUdf
Browse files Browse the repository at this point in the history
Author: Wenchen Fan <[email protected]>

Closes #6235 from cloud-fan/tmp and squashes the following commits:

8f16367 [Wenchen Fan] use private[this]
  • Loading branch information
cloud-fan authored and marmbrus committed May 18, 2015
1 parent 010a1c2 commit 56ede88
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -55,9 +55,9 @@ case class ScalaUdf(function: AnyRef, dataType: DataType, children: Seq[Expressi
}.foreach(println)
*/
val f = children.size match {
case 0 =>

private[this] val f = children.size match {
case 0 =>
val func = function.asInstanceOf[() => Any]
(input: Row) => {
func()
Expand Down Expand Up @@ -956,7 +956,7 @@ case class ScalaUdf(function: AnyRef, dataType: DataType, children: Seq[Expressi
}

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

}

0 comments on commit 56ede88

Please sign in to comment.