[GitHub] [pinot] Jackie-Jiang commented on a change in pull request #7174: Manually trigger PeriodicTask

2021-08-20 Thread GitBox
Jackie-Jiang commented on a change in pull request #7174: URL: https://github.com/apache/pinot/pull/7174#discussion_r693266380 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/periodictask/ControllerPeriodicTask.java ## @@ -55,14 +57,25 @@

[GitHub] [pinot] Jackie-Jiang commented on a change in pull request #7174: Manually trigger PeriodicTask

2021-08-19 Thread GitBox
Jackie-Jiang commented on a change in pull request #7174: URL: https://github.com/apache/pinot/pull/7174#discussion_r692568185 ## File path: pinot-core/src/main/java/org/apache/pinot/core/periodictask/BasePeriodicTask.java ## @@ -29,21 +33,31 @@ @ThreadSafe public abstract

[GitHub] [pinot] Jackie-Jiang commented on a change in pull request #7174: Manually trigger PeriodicTask

2021-08-19 Thread GitBox
Jackie-Jiang commented on a change in pull request #7174: URL: https://github.com/apache/pinot/pull/7174#discussion_r692560901 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/periodictask/ControllerPeriodicTask.java ## @@ -54,14 +55,27 @@

[GitHub] [pinot] Jackie-Jiang commented on a change in pull request #7174: Manually trigger PeriodicTask

2021-08-18 Thread GitBox
Jackie-Jiang commented on a change in pull request #7174: URL: https://github.com/apache/pinot/pull/7174#discussion_r690817077 ## File path: pinot-core/src/main/java/org/apache/pinot/core/periodictask/BasePeriodicTask.java ## @@ -29,21 +33,31 @@ @ThreadSafe public abstract

[GitHub] [pinot] Jackie-Jiang commented on a change in pull request #7174: Manually trigger PeriodicTask

2021-08-17 Thread GitBox
Jackie-Jiang commented on a change in pull request #7174: URL: https://github.com/apache/pinot/pull/7174#discussion_r690817077 ## File path: pinot-core/src/main/java/org/apache/pinot/core/periodictask/BasePeriodicTask.java ## @@ -29,21 +33,31 @@ @ThreadSafe public abstract

[GitHub] [pinot] Jackie-Jiang commented on a change in pull request #7174: Manually trigger PeriodicTask

2021-08-16 Thread GitBox
Jackie-Jiang commented on a change in pull request #7174: URL: https://github.com/apache/pinot/pull/7174#discussion_r689779004 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/ControllerMessageHandlerFactory.java ## @@ -0,0 +1,106 @@ +/** + * Licensed

[GitHub] [pinot] Jackie-Jiang commented on a change in pull request #7174: Manually trigger PeriodicTask

2021-08-16 Thread GitBox
Jackie-Jiang commented on a change in pull request #7174: URL: https://github.com/apache/pinot/pull/7174#discussion_r689770534 ## File path: pinot-controller/src/main/java/org/apache/pinot/controller/ControllerMessageHandlerFactory.java ## @@ -0,0 +1,106 @@ +/** + * Licensed

[GitHub] [pinot] Jackie-Jiang commented on a change in pull request #7174: Manually trigger PeriodicTask

2021-08-16 Thread GitBox
Jackie-Jiang commented on a change in pull request #7174: URL: https://github.com/apache/pinot/pull/7174#discussion_r689770125 ## File path: pinot-common/src/main/java/org/apache/pinot/common/messages/RunPeriodicTaskMessage.java ## @@ -0,0 +1,69 @@ +/** + * Licensed to the