AngersZhuuuu commented on pull request #32176: URL: https://github.com/apache/spark/pull/32176#issuecomment-820439084
> > Do we need to change typed construct too? > > Yep. Need to change this too. I just started from the small change in `visitInterval` but this already leads to significant changes in tests. Yea, after this done, we can start that part. One thing confused me a lot. Why we not implement DayTimeInterval/YearMonthInterval first but add a CalendarInterval. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
