tarak271 commented on code in PR #4653: URL: https://github.com/apache/hive/pull/4653#discussion_r1316736309
########## ql/src/java/org/apache/hadoop/hive/ql/udf/generic/AbstractGenericUDFArrayBase.java: ########## @@ -34,6 +35,7 @@ public abstract class AbstractGenericUDFArrayBase extends GenericUDF { static final int ARRAY_IDX = 0; + static final int ELEMENT_IDX = 1; Review Comment: Yes, keeping only checkValueAndListElementTypes(arrayElementOI, valueOI, elementIndex) method with index argument -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org