For comments.

Author: viirya

python/pyspark/sql/functions.py

@@ -232,7 +232,7 @@ def approxCountDistinct(col, rsd=None):
 
 
 @ignore_unicode_prefix
-@since(1.4)
+@since(1.5)
 def bin(col):
     """Returns the string representation of the binary value of the given column.
 

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

@@ -234,7 +234,7 @@ case class Bin(child: Expression)
 
   override def genCode(ctx: CodeGenContext, ev: GeneratedExpressionCode): String = {
     defineCodeGen(ctx, ev, (c) =>
-      s"org.apache.spark.unsafe.types.UTF8String.fromString(java.lang.Long.toBinaryString($c))")
+      s"${ctx.stringType}.fromString(java.lang.Long.toBinaryString($c))")
   }
 }
 

sql/core/src/main/scala/org/apache/spark/sql/functions.scala

@@ -886,7 +886,7 @@ object functions {
    * column. For example, bin("12") returns "1100".
    *
    * @group math_funcs
-   * @since 1.4.0
+   * @since 1.5.0
    */
   def bin(e: Column): Column = Bin(e.expr)
 
@@ -895,7 +895,7 @@ object functions {
    * column. For example, bin("12") returns "1100".
    *
    * @group math_funcs
-   * @since 1.4.0
+   * @since 1.5.0
    */
   def bin(columnName: String): Column = bin(Column(columnName))
 

sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala

@@ -17,8 +17,6 @@
 
 package org.apache.spark.sql
 
-import java.lang.{Long => JLong}
-
 import org.apache.spark.sql.TestData._
 import org.apache.spark.sql.functions._
 import org.apache.spark.sql.types._