Re: Introduce FORMAT clause to CAST with SQL:2016 datetime patterns

2019-04-02 Thread Gabor Kaszab
Thanks for the feedback! As I haven't received any comments recently and I hope I have addresses the previous ones I'll advance to the next step and open the related jiras for both Spark and Hive. Cheers, Gabor On Thu, Mar 21, 2019 at 12:00 PM Gabor Kaszab wrote: > Thanks for the quick feedbac

Re: Introduce FORMAT clause to CAST with SQL:2016 datetime patterns

2019-03-21 Thread Gabor Kaszab
Thanks for the quick feedbacks, Maciej and Shawn! Maciej: The concern about confusing users with supporting multiple datetime patterns is a valid one. The cleanest way to introduce SQL:2016 patterns would be to drop the existing pattern support (SimpleDateFormat in case of Impala) and replace it w

Re: Introduce FORMAT clause to CAST with SQL:2016 datetime patterns

2019-03-20 Thread Maciej Szymkiewicz
One concern here is introduction of second formatting convention. This can not only cause confusion among users, but also result in some hard to spot bugs, when wrong format, with different meaning, is used. This is already a problem for Python and R users, with week year and months / minutes mixu

Introduce FORMAT clause to CAST with SQL:2016 datetime patterns

2019-03-20 Thread Gabor Kaszab
Hey Hive and Spark communities, [dev@impala in cc] I'm working on an Impala improvement to introduce the FORMAT clause within CAST() operator and to implement ISO SQL:2016 datetime pattern support for this new FORMAT clause: https://issues.apache.org/jira/browse/IMPALA-4018 One example of the new