Open cosmic-chichu opened 10 months ago
@gianm would really appreciate your help with reviewing and merging this change?
It looks like the patch replaces the short form of table and intervals with long form, but, I'm wondering why this is necessary?
Since the short form produces a query that is not parseable using other Druid clients such as go-druid. Keeping it in the long form helps with disambiguation.
To maintain parity between druid clients such as go-druid and pydruid we need to express the intervals and datasource explicitly or in the fully expanded native query form.