bananaaggle commented on pull request #11262: URL: https://github.com/apache/druid/pull/11262#issuecomment-844638244
> > > ### 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, users can query movingAverage through router. One line code can break this limit, but it is not very graceful because router not use QuerySegmentWalker.class. > > Thanks for pointing out this. Yes, we need to make some code change to support moving average running from router. This is not what this PR aims for. I'm planning to do it later or if you're interested in this feature, you could take it. Last June I do this adjust on my cluster for using moving average for analysts. It works, but not graceful. Do you think it is a suitable idea to solve this problem? If you think it's OK, I can test it on master branch and open PR. It's very easy and I'm glad to do this. -- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
