Thanks Kevin!
On 4/24/18, 7:43 PM, "Kevin Risden" <[email protected]> wrote: Created CALCITE-2279 Kevin Risden On Tue, Apr 24, 2018 at 1:33 PM, Kevin Risden <[email protected]> wrote: > No JIRA yet. Will create one for this. > > Kevin Risden > > On Tue, Apr 24, 2018, 09:31 Jesus Camacho Rodriguez <[email protected]> > wrote: > >> Hi Kevin, >> >> Thanks for the feedback. Is there a JIRA for this case? It seems it is a >> bug indeed. >> >> -Jesús >> >> >> On 4/19/18, 8:06 PM, "Kevin Risden" <[email protected]> wrote: >> >> I was looking into Error Prone [1] checking for Calcite and it found >> what >> looks like a bug in Druid Adapter. The output is as follows >> >> DruidJsonFilter.java:[324,9] [IdentityBinaryExpression] A binary >> expression >> > where both operands are the same is usually incorrect; the value of >> this >> > expression is equivalent to `lhs.getType().getFamily() == >> > SqlTypeFamily.NUMERIC`. >> > [ERROR] (see >> > http://errorprone.info/bugpattern/IdentityBinaryExpression) >> >> >> The DruidJsonFilter [2] has left and right hand the exact same. >> Raising >> awareness here before going to JIRA. >> >> [1] http://errorprone.info/ >> [2] >> https://github.com/apache/calcite/blob/master/druid/src/ >> main/java/org/apache/calcite/adapter/druid/DruidJsonFilter.java#L323 >> >> Kevin Risden >> >> >> >>
