You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/04/15 13:17:09 UTC

[GitHub] [spark] MaxGekk commented on pull request #32176: [WIP][SPARK-35076][SQL] Parse interval literals as ANSI intervals

MaxGekk commented on pull request #32176:
URL: https://github.com/apache/spark/pull/32176#issuecomment-820414548


   In this PR, I enabled parsing literals as ANSI intervals instead of the existing CalendarIntervalType. This affects many tests in fact. @cloud-fan @yaooqinn @AngersZhuuuu @HyukjinKwon @maropu @dongjoon-hyun Could you help me to review the changes, please. Maybe need to enable legacy interval in some places. This PR is still a draft - need to fix much more tests.


-- 
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org