[GitHub] [druid] paul-rogers commented on pull request #13793: Add validation for aggregations on __time

["paul-rogers (via GitHub)" <gi...@apache.org>]

paul-rogers commented on PR #13793:
URL: https://github.com/apache/druid/pull/13793#issuecomment-1459114229

   After some offline discussions, it was felt that fixing this issue (by allowing `LATEST(x)` with MSQ) is unnecessary and overly complex. Instead, the original goal is wanted instead (disallow `LATEST(x)` in MSQ statements that do not finalize results.) Adding `LATEST(x)` support can be seen as a feature request for another time.
   
   The simplest solution is to disallow `LATEST(.)` if the engine is MSQ. There are finer points, but we can use the crude-but-effective solution for now.


-- 
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: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org