You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/02/19 18:53:48 UTC

[GitHub] [druid] jihoonson commented on a change in pull request #9358: Does not support Post-Aggregations

jihoonson commented on a change in pull request #9358: Does not support Post-Aggregations
URL: https://github.com/apache/druid/pull/9358#discussion_r381476331
 
 

 ##########
 File path: docs/development/extensions-contrib/materialized-view.md
 ##########
 @@ -134,3 +134,4 @@ There are 2 parts in a view query:
 |query	|The real query of this `view` query. The real query must be [groupBy](../../querying/groupbyquery.md), [topN](../../querying/topnquery.md), or [timeseries](../../querying/timeseriesquery.md) type.|yes|
 
 **Note that Materialized View is currently designated as experimental. Please make sure the time of all processes are the same and increase monotonically. Otherwise, some unexpected errors may happen on query results.**
+**Note Does not support Post-Aggregations**
 
 Review comment:
   Hi @ritik02, thank you for your contribution! Would you please rephrase to "**Note that the Materialized View does not support Post-Aggregations**"?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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