You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2020/12/30 00:12:28 UTC

[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #6395: Make minion tasks pluggable via reflection

Jackie-Jiang opened a new pull request #6395:
URL: https://github.com/apache/incubator-pinot/pull/6395


   ## Description
   Introduces 3 annotations to plugin minion components:
   - TaskGenerator: For task generators to be plugged on controller side
   - TaskExecutorFactory: For task executor factories to be plugged on minion side
   - EventObserverFactory: For event observer factories to be plugged on minion side
   
   Annotated classes will automatically be plugged into the controller and minion.
   
   ## Release Notes
   In order to make classes auto-pluggable, the following interfaces have been changed:
   - PinotTaskGenerator
   - PinotTaskExecutorFactory
   - MinionEventObserverFactory
   
   Customized implementation of these interfaces need to be modified accordingly.


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


[GitHub] [incubator-pinot] Jackie-Jiang merged pull request #6395: Make minion tasks pluggable via reflection

Posted by GitBox <gi...@apache.org>.
Jackie-Jiang merged pull request #6395:
URL: https://github.com/apache/incubator-pinot/pull/6395


   


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


[GitHub] [incubator-pinot] Jackie-Jiang commented on pull request #6395: Make minion tasks pluggable via reflection

Posted by GitBox <gi...@apache.org>.
Jackie-Jiang commented on pull request #6395:
URL: https://github.com/apache/incubator-pinot/pull/6395#issuecomment-752282798


   @snleee Please take a look and see if the interface change is okay for the LinkedIn internal tasks


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