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 2021/05/20 01:54:24 UTC

[GitHub] [druid] bananaaggle commented on pull request #11262: Improve doc of movingAverage

bananaaggle commented on pull request #11262:
URL: https://github.com/apache/druid/pull/11262#issuecomment-844621303


   > ### Description
   > There're some questions about how to enable movingAverage on brokers(eg #11086), this PR improves the doc to make operational steps more directive and also adds a limitation that movingAverage is not supported by druid router.
   > 
   > This PR has:
   > 
   > * [x]  been self-reviewed.
   
   Actually, add  binder.bind(c.class).to(FakeQuerySegmentWalker.class).in(LazySingleton.class)  to  CliRouter.java, user can query movingAverage through router. One line code can break this limit, but it is not very graceful because router not use QuerySegmentWalker.class.


-- 
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



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