Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19598#discussion_r148931423
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeExtractors.scala
---
@@ -242,9 +243,11 @@ case class GetArrayItem(child: Expression, ordinal:
Expression)
override def doGenCode(ctx: CodegenContext, ev: ExprCode): ExprCode = {
nullSafeCodeGen(ctx, ev, (eval1, eval2) => {
val index = ctx.freshName("index")
+ val nullCheck = if
(!child.dataType.asInstanceOf[ArrayType].containsNull) ""
+ else s" || $eval1.isNullAt($index)"
--- End diff --
nit:
```
if {
} else {
}
```
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
