You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2021/08/17 22:53:09 UTC
[GitHub] [superset] matthewphua opened a new issue #16302: Druid Time Series Error
matthewphua opened a new issue #16302:
URL: https://github.com/apache/superset/issues/16302
A Druid Error is generated when creating any time series chart. There error is cause by this line:
`WHERE FLOOR("__time" TO QUARTER) = '2015-07-01T00:00:00.000Z'`
### Expected results
An expected line chart should appear.
### Actual results
A Druid error appears:
<img width="1786" alt="Screen Shot 2021-08-17 at 3 49 37 PM" src="https://user-images.githubusercontent.com/85198947/129810739-b676a493-5b45-4c52-b2ef-7c9090e4721b.png">
<img width="1792" alt="Screen Shot 2021-08-17 at 3 49 54 PM" src="https://user-images.githubusercontent.com/85198947/129810763-20cf744b-550d-46b9-9a00-ddfee8caf215.png">
#### How to reproduce the bug
1. Use a Druid DB
2. Create a line chart, enter a time grain (optional, I think with or without a time grain, it causes an error)
3. Fill out all required fields
4. Run and See error
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] zhaoyongjie commented on issue #16302: [Explore]Druid Time Series Error
Posted by GitBox <gi...@apache.org>.
zhaoyongjie commented on issue #16302:
URL: https://github.com/apache/superset/issues/16302#issuecomment-903778586
Hi @matthewphua @junlincc, this issue exists in all databases/data warehouse with Calcite as the SQL parser(Druid/Kylin/Drill), because Calcite doesn't make an implicit conversion for difference data type.
the SQL should be
```
WHERE FLOOR("__time" TO QUARTER) = case('2015-07-01T00:00:00.000Z', timestamp)
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] zhaoyongjie edited a comment on issue #16302: [Explore]Druid Time Series Error
Posted by GitBox <gi...@apache.org>.
zhaoyongjie edited a comment on issue #16302:
URL: https://github.com/apache/superset/issues/16302#issuecomment-903778586
Hi @matthewphua @junlincc, this issue exists in all databases/data warehouse with Calcite as the SQL parser(Druid/Kylin/Drill), because Calcite doesn't make an implicit conversion for difference data type.
the SQL should be
```
WHERE FLOOR("__time" TO QUARTER) = case('2015-07-01T00:00:00.000Z' as timestamp)
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] junlincc commented on issue #16302: Druid Time Series Error
Posted by GitBox <gi...@apache.org>.
junlincc commented on issue #16302:
URL: https://github.com/apache/superset/issues/16302#issuecomment-900810316
it can be reproduced in table chart. but not nvd3. @matthewphua thank you for reporting! 🙏
@zhaoyongjie let's take a look? thanks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] betodealmeida commented on issue #16302: [Explore]Druid Time Series Error
Posted by GitBox <gi...@apache.org>.
betodealmeida commented on issue #16302:
URL: https://github.com/apache/superset/issues/16302#issuecomment-923446329
A few comments on this:
- If you remove `__time` from the group bys the query works. You shouldn't add `__time` to the group bys because you're already using it for the time grain.
- Druid already supports joins, if we turn it on for the DB engine spec then the query also works.
- We should still fix 2-phase queries so that the predicate for the top N works regardless of types.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] junlincc commented on issue #16302: Druid Time Series Error
Posted by GitBox <gi...@apache.org>.
junlincc commented on issue #16302:
URL: https://github.com/apache/superset/issues/16302#issuecomment-900810316
it can be reproduced in table chart. but not nvd3. @matthewphua thank you for reporting! 🙏
@zhaoyongjie let's take a look? thanks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] zhaoyongjie edited a comment on issue #16302: [Explore]Druid Time Series Error
Posted by GitBox <gi...@apache.org>.
zhaoyongjie edited a comment on issue #16302:
URL: https://github.com/apache/superset/issues/16302#issuecomment-903778586
Hi @matthewphua @junlincc, this issue exists in all databases/data warehouse with Calcite as the SQL parser(Druid/Kylin/Drill), because Calcite doesn't make an implicit conversion for difference data type.
the SQL should be
```
WHERE FLOOR("__time" TO QUARTER) = cast('2015-07-01T00:00:00.000Z' as timestamp)
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org