Github user ueshin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21208#discussion_r187856418
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
 ---
    @@ -1468,3 +1468,149 @@ case class Flatten(child: Expression) extends 
UnaryExpression {
     
       override def prettyName: String = "flatten"
     }
    +
    +/**
    + * Returns the array containing the given input value (left) count (right) 
times.
    + */
    +@ExpressionDescription(
    +  usage = "_FUNC_(element, count) - Returns the array containing element 
count times.",
    +  examples = """
    +    Examples:
    +      > SELECT _FUNC_('123', 2);
    +       ['123', '123']
    +  """)
    +case class ArrayRepeat(left: Expression, right: Expression)
    +  extends BinaryExpression with ExpectsInputTypes {
    +
    +  private val MAX_ARRAY_LENGTH = ByteArrayMethods.MAX_ROUNDED_ARRAY_LENGTH
    +
    +  override def dataType: ArrayType = ArrayType(left.dataType, 
left.nullable)
    +
    +  override def inputTypes: Seq[AbstractDataType] = Seq(AnyDataType, 
IntegerType)
    +
    +  override def nullable: Boolean = right.nullable
    +
    +  override def eval(input: InternalRow): Any = {
    +    val count = right.eval(input)
    +    if (count == null) {
    +      null
    +    } else {
    +      if (count.asInstanceOf[Int] > MAX_ARRAY_LENGTH) {
    +        throw new RuntimeException(s"Unsuccessful try to create array with 
$count elements" +
    --- End diff --
    
    nit: need a space after `elements`.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to