dennis zhuang
dennis zhuang
@NiwakaDev Hi, do you have time to resolve these comments? I think this PR is almost done, looks great, thank you.
> I have a few concerns about adding a new `within` grammar. It brings more burden of maintaining the sqlparser lib, and is not generic enough (I did some search...
The range query is worth the maintenance cost due to its usefulness and elegance, but I agree with you that `within` is unnecessary. @waynexia Could you refactor this PR, @NiwakaDev?...
> @killme2008 @waynexia > > > Please implement it as an SQL function. Most of the work is already done. Thanks! > > The new following approach: > > 1....
@NiwakaDev Sorry to disturb you, but do you have time to review this PR? It's nearly finished and would benefit our users. If not, I would love to help.
> @killme2008 > > Sorry for the delay. March was busy😅 > > I can finally start working on this PR this week if you don't mind. If I can't...
@zhongzc Do we have a plan to fix it?
I think we must deliver this PR in v0.9.1, so please take a look @sunng87 @shuiyisong @zhongzc
> Incompatible changes added > > 1. Delete `date`, `epoch` processor > 2. add `timestamp` process, which is a combination of date and epoch. It can parse time strings as...
What's the progress of this PR? @shuiyisong @paomian