HyukjinKwon commented on a change in pull request #32340: URL: https://github.com/apache/spark/pull/32340#discussion_r751925908
########## File path: sql/catalyst/src/main/java/org/apache/spark/sql/vectorized/ArrowColumnVector.java ########## @@ -172,6 +176,10 @@ public ArrowColumnVector(ValueVector vector) { } } else if (vector instanceof NullVector) { accessor = new NullAccessor((NullVector) vector); + } else if (vector instanceof IntervalYearVector) { + accessor = new IntervalYearAccessor((IntervalYearVector) vector); + } else if (vector instanceof IntervalDayVector) { + accessor = new IntervalDayAccessor((IntervalDayVector) vector); Review comment: and, `YearMonthIntervalType` is mapped to [`java.time.Period`](https://docs.oracle.com/javase/8/docs/api/java/time/Period.html) which is a calendar instance: > A date-based amount of time in the ISO-8601 calendar system, such as '2 years, 3 months and 4 days'. So `YearMonthIntervalType` seems fine. -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org