[GitHub] [incubator-pinot] jasonyanwenl commented on pull request #5769: [TE] add anomaly detection as a service - Phase 1

[GitBox <gi...@apache.org>]

jasonyanwenl commented on pull request #5769:
URL: https://github.com/apache/incubator-pinot/pull/5769#issuecomment-666769896


   @suvodeep-pyne Thanks for your feedback! For each of your points:
   * I think those pieces of code are related to each other so currently there is no plan to separate them into multiple RPs. In addition, there is a global flag in the configuration file that can turn on/off this feature. The next time I send a PR, I will keep this in mind.
   * The resource will keep polling the task status to make sure this service is synchronous from the user side.
   * The resource will not directly interact with the task and it will not directly call backend method. The resource will only create a task and save it in to DB. The worker thread in the backend will asynchronously retrieve it.


----------------------------------------------------------------
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@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org