Re: [Flink-SQL] FLOORing OR CEILing a DATE or TIMESTAMP to WEEK uses Thursdays as week start

[Leonard Xu]

Hi, Sebastián Ramírez Magrí 
(Sorry for wrong name in above mail)

Flink follows old version calcite’s behaviour which lead to the wrong behavior. 
snd Julian is right that calcite has corrected FLOOR and CEIL functions in 
CALCITE-3412, Flink has upgraded  calcite to 1.26 version which contains the 
patch, what we need is only to adapt it in Flink code, I’d like to make this as 
a part of FLIP-162 and fix it soon.

Thanks Julian and Timo for the reminder.


Best,
Leonard

> 在 2021年3月4日，12:14，Leonard Xu <xbjt...@gmail.com> 写道：
> 
> Hi, Jaffe
> 
> Flink follows old version calcite’s behaviour which lead to the wrong 
> behavior. snd Julian is right that calcite has corrected FLOOR and CEIL 
> functions in CALCITE-3412, Flink has upgraded  calcite to 1.26 version which 
> contains the patch, what we need is only to adapt it in Flink code, I’d like 
> to make this as a part of FLIP-162 and fix it soon.
> 
> Thanks Julian and Timo for the reminder.
> 
> 
> Best,
> Leonard