Stream-stream joins

2018-07-11 Thread Piotr Nowojski
this feature? Is [3] the way to go in the foreseeable future? If so, is anyone working on it at the moment? Piotr Nowojski [1] https://docs.google.com/document/d/1KaAkPZjWFeu-ffrC9FhYuxE6CIKsatHTTxyrxSBR8Sk/edit?ts=5b45fea6#heading=h.avxnnym51mbe <https://docs.google.com/document/d/1KaA

Re: Stream-stream joins

2018-07-15 Thread Piotr Nowojski
ro rather than a > table function.> > > Julian> > > > On Jul 11, 2018, at 6:55 AM, Piotr Nowojski > > wrote:> > > > > > Hi,> > > > > > In Flink we are now discussing how to implement “enrichment joins” (stream &g

VARCHAR literals

2018-09-06 Thread Piotr Nowojski
Hi, We have small problem with CHAR type in Flink. Officially we do not support it and all input/output columns are of type VARCHAR. Because of that, nobody has ever thought about CHAR semantic (for example correctly handling padding in comparisons or other functions). However this collides wit

Re: VARCHAR literals

2018-09-06 Thread Piotr Nowojski
We know about this switch. Unfortunately it only solves/hides one of the problems. > On 6 Sep 2018, at 20:02, Julian Hyde wrote: > > Does https://issues.apache.org/jira/browse/CALCITE-2321 > <https://issues.apache.org/jira/browse/CALCITE-2321> help? > > >> On

Re: VARCHAR literals

2018-09-10 Thread Piotr Nowojski
Julian, would you be ok if we provided a contribution containing a configurable switch for string literal type? Piotrek > On 6 Sep 2018, at 20:47, Piotr Nowojski wrote: > > We know about this switch. Unfortunately it only solves/hides one of the > problems. > >> On

Re: VARCHAR literals

2018-09-11 Thread Piotr Nowojski
t’s not done properly Calcite will > be picking up the pieces for years. > > Julian > > >> On Sep 10, 2018, at 1:26 AM, Piotr Nowojski wrote: >> >> Julian, would you be ok if we provided a contribution containing a >> configurable switch for string literal ty

Intention behind RelNode.accept(RexShuttle)

2019-01-31 Thread Piotr Nowojski
node. So can we leave this method with the default implementation `return this;` as Aggregate node does it? Thanks, Piotr Nowojski

Re: Intention behind RelNode.accept(RexShuttle)

2019-02-04 Thread Piotr Nowojski
s not > contain row expressions (RexNode). If the node you introduce uses row > expressions then it makes sense to apply the shuttle to every expression. > > Best, > Stamatis > > Στις Πέμ, 31 Ιαν 2019 στις 5:39 μ.μ., ο/η Piotr Nowojski < > pi...@da-platform.com> έγραψε

Re: Intention behind RelNode.accept(RexShuttle)

2019-02-05 Thread Piotr Nowojski
of a RelNode are > sub-types of RexNode, RexShuttle processes them, otherwise it ignores them. > But pragmatically, as long as what you do doesn’t break the tests, it’s OK. > >> On Feb 4, 2019, at 1:05 AM, Piotr Nowojski wrote: >> >> Hi Stamatis, >> >>