[GitHub] [incubator-druid] julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException

2019-09-20 Thread GitBox
julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException URL: https://github.com/apache/incubator-druid/issues/7953#issuecomment-533692535 It's in the comments above. CALCITE-3144.

[GitHub] [incubator-druid] julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException

2019-07-11 Thread GitBox
julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException URL: https://github.com/apache/incubator-druid/issues/7953#issuecomment-510700896 @gianm Now we've established that AggregateExpandDistinctAggregatesRule is at fault, c

[GitHub] [incubator-druid] julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException

2019-07-11 Thread GitBox
julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException URL: https://github.com/apache/incubator-druid/issues/7953#issuecomment-510699278 @gianm That sounds very plausible. I honestly don't remember why I converted `CO

[GitHub] [incubator-druid] julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException

2019-07-01 Thread GitBox
julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException URL: https://github.com/apache/incubator-druid/issues/7953#issuecomment-507523316 I tried to fix this in Calcite. I needed to modify the "A1" case in the rule to exclud

[GitHub] [incubator-druid] julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException

2019-06-24 Thread GitBox
julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException URL: https://github.com/apache/incubator-druid/issues/7953#issuecomment-505130432 I've logged https://issues.apache.org/jira/browse/CALCITE-3144 to track. -

[GitHub] [incubator-druid] julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException

2019-06-24 Thread GitBox
julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException URL: https://github.com/apache/incubator-druid/issues/7953#issuecomment-505128442 I see, added in #4360. If you'd like to contribute that back to Calcite, we'd be glad

[GitHub] [incubator-druid] julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException

2019-06-24 Thread GitBox
julianhyde commented on issue #7953: Exact distinct-COUNT with complex expression (CASE, IN) throws NullPointerException URL: https://github.com/apache/incubator-druid/issues/7953#issuecomment-505121858 I agree, and the crack is probably in Calcite. `IN` is a good place to look, because i