bolkedebruin commented on issue #4114: Fix using undefined DruidMetrics for 
limit_spec
URL: 
https://github.com/apache/incubator-superset/pull/4114#issuecomment-354942065
 
 
   Ah that looks great! I?m aware of the effort to create a sql interface to 
Druid. That should put adoption on steroids I guess, but it will take awhile 
before that is deemed stable. 
   
   So I think you?re right on how the issue is triggered from MapBox, due to 
the absence of a defined metric in the UI superset defaults to a set of 
pre-defined ones which 1) are not in the query and thus not known to Druid and 
2) not normalize (as done at other places) to just it?s value hence being 
unserializable. 
   
   The pr uses the dimension as per Druid spec if a metric is absent. 
   
   So setting ?is_timeseries? to false probably fixes the symptom but will hide 
the bug. 
     

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to