ihuzenko commented on a change in pull request #1933: DRILL-7473: Parquet
reader failed to get field of repeated map
URL: https://github.com/apache/drill/pull/1933#discussion_r358173016
##########
File path:
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/validate/BatchValidator.java
##########
@@ -293,14 +292,20 @@ private void validateVector(String name, ValueVector
vector) {
// structure to check.
} else if (vector instanceof BaseRepeatedValueVector) {
validateRepeatedVector(name, (BaseRepeatedValueVector) vector);
- } else if (vector instanceof RepeatedMapVector) {
- validateRepeatedMapVector(name, (RepeatedMapVector) vector);
+ } else if (vector instanceof AbstractRepeatedMapVector) {
Review comment:
Could you please clarify your comment below, for example
>Checking ```AbstractRepeatedMapVector``` includes DictVector, in order to
avoid NPE while getting value from ```NullReader```. The ```NullReader``` may
be returned when index points to the absent value in dict array, example query
```SELECT dict_array[3].keyInDict ... ``` .
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
