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