Jackie-Jiang commented on a change in pull request #3819: Refactor periodic task URL: https://github.com/apache/incubator-pinot/pull/3819#discussion_r255828508
########## File path: pinot-core/src/main/java/org/apache/pinot/core/periodictask/BasePeriodicTask.java ########## @@ -42,9 +61,125 @@ public long getInitialDelayInSeconds() { return _initialDelayInSeconds; } + /** + * Returns the status of the {@code started} flag. This flag will be set after calling {@link #start()}, and reset + * after calling {@link #stop()}. + */ + public final boolean isStarted() { + return _started; + } + + /** + * Returns the status of the {@code running} flag. This flag will be set during the task execution. + */ + public final boolean isRunning() { + return _running; + } + + /** + * {@inheritDoc} + * <p> + * This method sets {@code started} flag to true. + */ @Override - public String getTaskName() { - return _taskName; + public final synchronized void start() { Review comment: No, the purpose of this synchronized is to avoid calling start() and stop() at the same time. Basically another level of protection. (start() and stop() are called in scheduler, not in the task thread) ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@pinot.apache.org For additional commands, e-mail: dev-h...@pinot.apache.org