Till Westmann has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/1218

Change subject: AbstractWork does not implement Runnable anymore
......................................................................

AbstractWork does not implement Runnable anymore

... and gets a comment on the expected behavior of run().

Change-Id: I53e77af9edc945fbea2a19a20b6d1f7568ac6b3a
---
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/work/AbstractWork.java
1 file changed, 7 insertions(+), 1 deletion(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/18/1218/1

diff --git 
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/work/AbstractWork.java
 
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/work/AbstractWork.java
index 9376370..4cf521f 100644
--- 
a/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/work/AbstractWork.java
+++ 
b/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/work/AbstractWork.java
@@ -20,7 +20,7 @@
 
 import java.util.logging.Level;
 
-public abstract class AbstractWork implements Runnable {
+public abstract class AbstractWork {
     public Level logLevel() {
         return Level.INFO;
     }
@@ -31,6 +31,12 @@
         return className.substring(className.lastIndexOf('.') + 1, endIndex);
     }
 
+    /**
+     * run is executed on a single thread that services the work queue. As a 
result run should never wait or block as
+     * this will deplay processing for the whole queue.
+     */
+    public abstract void run();
+
     @Override
     public String toString() {
         return getName();

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1218
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I53e77af9edc945fbea2a19a20b6d1f7568ac6b3a
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <[email protected]>

Reply via email to