tedyu commented on a change in pull request #35099:
URL: https://github.com/apache/spark/pull/35099#discussion_r778464386



##########
File path: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeExtractors.scala
##########
@@ -233,7 +233,7 @@ case class GetArrayStructFields(
 case class GetArrayItem(
     child: Expression,
     ordinal: Expression,
-    failOnError: Boolean = SQLConf.get.ansiFailOnElementNotExists)
+    failOnError: Boolean = 
SQLConf.get.getConf(SQLConf.ANSI_STRICT_INDEX_OPERATOR))

Review comment:
       @cloud-fan 
   I found the following in the log of failed test run:
   ```
   [info]   Cause: org.apache.spark.SparkNoSuchElementException: Key brother 
does not exist. If necessary set spark.sql.ansi.strictIndexOperator to false to 
bypass this error.
   ```
   I came to the same conclusion as @dongjoon-hyun pointed out above.
   I think we can restore to renaming ansiFailOnElementNotExists in my original 
PR.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to