Changeset 102 for java/main/src/main/java/com/framsticks/util/dispatching/BufferedDispatcher.java

Timestamp:

07/16/13 23:31:35 (11 years ago)

Author:

psniegowski

Message:

HIGHLIGHTS:

• add StackedJoinable? as a utility class to allow strict stack policy

for Joinables running

• add SimulatorConnector?
  • provide Experiment with single Simulator instance
  • checks loaded expdef
• fix bug with sockets closure
• improve task dispatching on startup (BufferedDispatcher?)
• draft Experiment implementation
• add NetLoadSaveLogic?
• add WorkPackageLogic?
• add PrimeExperiment? experiment class

CHANGELOG:
Add WorkPackageLogic? and classes representing prime experiment state.

Add classes for PrimeExperiment? state.

Extract single netload routine in Simulator.

Working netload with dummy content in PrimeExperiment?.

More development with NetLoadSaveLogic? and PrimeExperiment?.

Improvement around prime.

Improve BufferedDispatcher?.isActive logic.

Add prime-all.xml configuration.

Manual connecting to existing simulators from GUI.

Guard in SimulatorConnector? against expdef mismatch.

Guard against empty target dispatcher in BufferedDispatcher?.

Make BufferedDispatcher? a Dispatcher (and Joinable).

Minor improvements.

Done StackedJoinable?, improve Experiment.

Develop StackedJoinable?.

Add StackedJoinable? utility joinables controller.

Add dependency on apache-commons-lang.

Add ready ListChange? on Simulators.

Improve hints in ListChange?.

Several improvements.

Found bug with dispatching in Experiment.

Minor improvements.

Fix bug with early finishing Server.

Many changes in Dispatching.

Fix bug with connection.

Do not obfuscate log with socket related exceptions.

Add SocketClosedException?.

Add SimulatorConnector?.

Work out conception of experiment composing of logics building blocks.

Rename SinkInterface? to Sink.

Move saving of Accesses into AccessOperations?.

Some improvements to Experiment.

Improve joinables.

Fix issue with joinables closing.

Add direct and managed consoles to popup menu.

File:

• java/main/src/main/java/com/framsticks/util/dispatching/BufferedDispatcher.java (modified) (5 diffs)

java/main/src/main/java/com/framsticks/util/dispatching/BufferedDispatcher.java

@@ -2 +2 @@
 
 
-public class BufferedDispatcher<C> {
+import org.apache.logging.log4j.Logger;
+import org.apache.logging.log4j.LogManager;
+
+import com.framsticks.util.FramsticksException;
+import com.framsticks.util.lang.Strings;
+// import static com.framsticks.util.dispatching.AbstractJoinable.wrap;
+
+public class BufferedDispatcher<C> extends AbstractJoinable implements Dispatcher<C>, JoinableParent {
+        private static final Logger log = LogManager.getLogger(BufferedDispatcher.class);
 
         protected final RunnableQueue<C> queue = new RunnableQueue<>();
@@ -8 +16 @@
         protected Dispatcher<C> targetDispatcher;
 
-        protected boolean buffer = false;
+        protected boolean buffer = true;
+        protected Dispatcher<C> parent;
+
+        /**
+         * @param parent
+         */
+        public BufferedDispatcher(Dispatcher<C> parent) {
+                this.parent = parent;
+        }
 
         /**
@@ -21 +37 @@
          */
         public synchronized void setTargetDispatcher(Dispatcher<C> targetDispatcher) {
+                if (this.targetDispatcher != null) {
+                        throw new FramsticksException().msg("dispatcher is already set").arg("dispatcher", this.targetDispatcher).arg("in", this);
+                }
+                if (targetDispatcher == null) {
+                        throw new FramsticksException().msg("trying to set empty target dispatcher").arg("in", this);
+                }
+                log.debug("setting {} to {}", this, targetDispatcher);
                 this.targetDispatcher = targetDispatcher;
+                flushQueue();
         }
 
         public synchronized boolean isActive() {
-                if (targetDispatcher == null) {
+                if (targetDispatcher == null || buffer) {
                         return false;
                         // throw new FramsticksException().msg("no dispatcher is set for tree yet").arg("tree", this);
@@ -40 +64 @@
         }
 
+        protected void flushQueue() {
+                if (this.buffer || targetDispatcher == null) {
+                        return;
+                }
+                log.debug("flushing {} tasks in {}", queue.size(), this);
+                while (!queue.isEmpty()) {
+                        targetDispatcher.dispatch(queue.pollFirst());
+                }
+        }
+
         public synchronized void setBuffer(final boolean buffer) {
                 if (this.buffer == buffer) {
@@ -49 +83 @@
                 }
                 this.buffer = false;
-                while (!queue.isEmpty()) {
-                        targetDispatcher.dispatch(queue.pollFirst());
+                flushQueue();
+        }
+
+        @Override
+        public String toString() {
+                return parent + " -> " + Strings.toStringNullProof(targetDispatcher, "<null>");
+        }
+
+        public void createThreadIfNeeded() {
+                if (targetDispatcher != null) {
+                        return;
                 }
+                this.setTargetDispatcher(new Thread<C>().setName(parent.getName()));
+        }
+
+        @Override
+        public String getName() {
+                return parent + " buffered dispatcher";
+        }
+
+        @Override
+        protected void joinableStart() {
+                Dispatching.use(targetDispatcher, this);
+        }
+
+        @Override
+        protected void joinableInterrupt() {
+                Dispatching.drop(targetDispatcher, this);
+
+                finishJoinable();
+        }
+
+        @Override
+        protected void joinableFinish() {
 
         }
 
+        @Override
+        protected void joinableJoin() throws InterruptedException {
+                Dispatching.join(targetDispatcher);
+        }
 
+        @Override
+        public void childChangedState(Joinable joinable, JoinableState state) {
+                proceedToState(state);
+        }
 }