]> gerrit.simantics Code Review - simantics/platform.git/blobdiff - bundles/org.simantics.db.impl/src/org/simantics/db/impl/query/AsyncReadEntry.java
Fixed multiple issues causing dangling references to discarded queries
[simantics/platform.git] / bundles / org.simantics.db.impl / src / org / simantics / db / impl / query / AsyncReadEntry.java
index 6f681056dd07bea4e755900906af540bcb88772d..2fdb43d113332e4f33b6b131685fd8c80c112b22 100644 (file)
-/*******************************************************************************\r
- * Copyright (c) 2007, 2010 Association for Decentralized Information Management\r
- * in Industry THTH ry.\r
- * All rights reserved. This program and the accompanying materials\r
- * are made available under the terms of the Eclipse Public License v1.0\r
- * which accompanies this distribution, and is available at\r
- * http://www.eclipse.org/legal/epl-v10.html\r
- *\r
- * Contributors:\r
- *     VTT Technical Research Centre of Finland - initial API and implementation\r
- *******************************************************************************/\r
-package org.simantics.db.impl.query;\r
-\r
-import java.util.ArrayList;\r
-\r
-import org.simantics.db.AsyncReadGraph;\r
-import org.simantics.db.impl.DebugPolicy;\r
-import org.simantics.db.impl.graph.ReadGraphImpl;\r
-import org.simantics.db.procedure.AsyncProcedure;\r
-import org.simantics.db.request.AsyncRead;\r
-\r
-final public class AsyncReadEntry<T> extends CacheEntryBase {\r
-\r
-    protected AsyncRead<T> request;\r
-\r
-    public AsyncReadEntry(AsyncRead<T> request) {\r
-       this.request = request;\r
-       if(DebugPolicy.QUERY_STATE) System.out.println("[QUERY STATE]: created " + this);\r
-    }\r
-\r
-    @Override\r
-    int makeHash() {\r
-       return request.hashCode();\r
-    }\r
-    \r
-    @Override\r
-    public Object getOriginalRequest() {\r
-        return request;\r
-    }\r
-    \r
-    @Override\r
-    public void discard() {\r
-       super.discard();\r
-       //request = null;\r
-       setResult(null);\r
-    }\r
-    \r
-    final public void addOrSet(AsyncReadGraph graph, Object item) {\r
-\r
-       assert(isPending());\r
-        \r
-//        ArrayList<AsyncProcedure<T>> p = null;\r
-\r
-        synchronized(this) {\r
-               \r
-            setResult(item);\r
-               setReady();\r
-//            p = procs;\r
-//            procs = null;\r
-            \r
-        }\r
-\r
-//        if(p != null)\r
-//            for(AsyncProcedure<T> proc : p) {\r
-//             proc.execute(graph, (T)item);\r
-////                proc.first.execute(graph, (T)item);\r
-////                proc.second.dec();\r
-//            }\r
-        \r
-    }\r
-    \r
-    \r
-    public void except(AsyncReadGraph graph, Throwable t) {\r
-       \r
-       assert(isPending());\r
-        \r
-//        ArrayList<AsyncProcedure<T>> p = null;\r
-\r
-        synchronized(this) {\r
-               \r
-            except(t);\r
-////            p = procs;\r
-//            procs = null;\r
-            \r
-        }\r
-\r
-//        if(p != null)\r
-//            for(AsyncProcedure<T> proc : p) {\r
-//             proc.exception(graph, t);\r
-//            }\r
-       \r
-    }\r
-    \r
-    \r
-    @Override\r
-    final public Query getQuery() {\r
-       \r
-        return new Query() {\r
-\r
-                       @Override\r
-                       public void recompute(ReadGraphImpl graph, Object provider, CacheEntry entry) {\r
-                               \r
-                               QueryProcessor qp = (QueryProcessor)provider;\r
-\r
-                               final ReadGraphImpl parentGraph = ReadGraphImpl.forRecompute(entry, qp); \r
-\r
-                               try {\r
-\r
-                                   request.perform(parentGraph , new AsyncProcedure<T>() {\r
-\r
-                        @Override\r
-                        public void execute(AsyncReadGraph graph, T result) {\r
-                            addOrSet(graph, result);\r
-                        }\r
-                               \r
-                               @Override\r
-                               public void exception(AsyncReadGraph graph, Throwable t) {\r
-                                   except(t);\r
-                        }\r
-\r
-                    });\r
-\r
-                               } catch (Throwable t) {\r
-                                   except(t);\r
-                }\r
-                               \r
-                       }\r
-\r
-                       @Override\r
-                       public void removeEntry(QueryProcessor qp) {\r
-                       qp.asyncReadMap.remove(request);\r
-                       }\r
-\r
-                       @Override\r
-                       public int type() {\r
-                               return request.getFlags();\r
-                       }\r
-                       \r
-                       @Override\r
-                       public String toString() {\r
-                               if(request == null) return "DISCARDED";\r
-                               else if(isExcepted()) return request.toString() + " " + getResult();\r
-                               else return request.toString() + " " + statusOrException;\r
-                       }\r
-               \r
-        };\r
-        \r
-    }\r
-\r
-       public void performFromCache(ReadGraphImpl graph, Object provider, Object procedure) {\r
-               \r
-        AsyncProcedure<T> proc = (AsyncProcedure<T>)procedure;\r
-\r
-           if(isExcepted()) {\r
-            \r
-            try {\r
-                proc.exception(graph, (Throwable)getResult());\r
-            } catch (Throwable t) {\r
-                t.printStackTrace();\r
-            }\r
-            \r
-        } else {\r
-            \r
-            try {\r
-                proc.execute(graph, (T)getResult());\r
-            } catch (Throwable t) {\r
-                t.printStackTrace();\r
-            }\r
-            \r
-        }\r
-               \r
-       }\r
-\r
-       @Override\r
-       public String toString() {\r
-               if(isDiscarded()) return "DISCARDED " + request.toString();\r
-               else if(isExcepted()) return request.toString() + " " + getResult();\r
-               else return request.toString() + " " + statusOrException;\r
-       }\r
-\r
-}\r
+/*******************************************************************************
+ * Copyright (c) 2007, 2018 Association for Decentralized Information Management
+ * in Industry THTH ry.
+ * All rights reserved. This program and the accompanying materials
+ * are made available under the terms of the Eclipse Public License v1.0
+ * which accompanies this distribution, and is available at
+ * http://www.eclipse.org/legal/epl-v10.html
+ *
+ * Contributors:
+ *     VTT Technical Research Centre of Finland - initial API and implementation
+ *******************************************************************************/
+package org.simantics.db.impl.query;
+
+import java.util.Collection;
+
+import org.simantics.databoard.Bindings;
+import org.simantics.db.AsyncReadGraph;
+import org.simantics.db.DevelopmentKeys;
+import org.simantics.db.exception.DatabaseException;
+import org.simantics.db.impl.BlockingAsyncProcedure;
+import org.simantics.db.impl.graph.ReadGraphImpl;
+import org.simantics.db.impl.query.QueryProcessor.SessionTask;
+import org.simantics.db.procedure.AsyncProcedure;
+import org.simantics.db.request.AsyncRead;
+import org.simantics.utils.Development;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+final public class AsyncReadEntry<T> extends CacheEntryBase<AsyncProcedure<T>> implements AsyncProcedure<T>, IPending {
+
+    private static final Logger LOGGER = LoggerFactory.getLogger(AsyncReadEntry.class);
+
+    protected AsyncRead<T> id;
+    protected PendingTaskSupport pendingTaskSupport;
+
+    AsyncReadEntry(AsyncRead<T> request) {
+        this.id = request;
+               if (Development.DEVELOPMENT) {
+                       if(Development.<Boolean>getProperty(DevelopmentKeys.CACHE_ENTRY_STATE, Bindings.BOOLEAN)) {
+                       System.err.println("[QUERY STATE]: created " + this);
+                       }
+               }
+    }
+
+    @Override
+    int makeHash() {
+        return id.hashCode();
+    }
+
+    @Override
+    public Object getOriginalRequest() {
+        return id;
+    }
+
+    @Override
+    public void discard() {
+        super.discard();
+        setResult(null);
+    }
+
+    public void except(AsyncReadGraph graph, Throwable t) {
+
+        assert (isPending());
+
+        synchronized (this) {
+            except(t);
+        }
+
+    }
+
+    @Override
+    final public Query getQuery() {
+
+        return new Query() {
+
+            @Override
+            public void recompute(ReadGraphImpl graph) {
+
+                try {
+
+                    BlockingAsyncProcedure<T> proc = new BlockingAsyncProcedure(graph, AsyncReadEntry.this, new AsyncProcedure<T>() {
+
+                        @Override
+                        public void execute(AsyncReadGraph graph, T result) {
+                            setResult(result);
+                            setReady();
+                        }
+
+                        @Override
+                        public void exception(AsyncReadGraph graph, Throwable t) {
+                            except(t);
+                        }
+
+                    }, id, true);
+                    
+                    proc.performSync(id);
+
+                } catch (Throwable t) {
+                    except(t);
+                }
+
+            }
+
+            @Override
+            public void removeEntry(QueryProcessor qp) {
+                qp.cache.remove(AsyncReadEntry.this);
+            }
+
+            @Override
+            public int type() {
+                return id.getFlags();
+            }
+
+            @Override
+            public String toString() {
+                if (id == null)
+                    return "DISCARDED";
+                else if (isExcepted())
+                    return id.toString() + " " + getResult();
+                else
+                    return id.toString() + " " + statusOrException;
+            }
+
+        };
+
+    }
+
+    @Override
+    public Object performFromCache(ReadGraphImpl graph, AsyncProcedure<T> proc) {
+
+        if (isExcepted()) {
+
+            try {
+                proc.exception(graph, (Throwable) getResult());
+            } catch (Throwable t) {
+                LOGGER.error("performFromCache proc.exception failed", t);
+            }
+
+        } else {
+
+            try {
+                T result = (T) getResult();
+                proc.execute(graph, result);
+            } catch (Throwable t) {
+                LOGGER.error("performFromCache proc.execute failed", t);
+            }
+
+        }
+
+        return getResult();
+
+    }
+
+    public static <T> T computeForEach(ReadGraphImpl callerGraph, AsyncRead<T> request, AsyncReadEntry<T> entry,
+            AsyncProcedure<T> procedure_, boolean needsToBlock) throws DatabaseException {
+
+        BlockingAsyncProcedure<T> proc = new BlockingAsyncProcedure(callerGraph, entry, procedure_, request, needsToBlock);
+        if(needsToBlock) {
+            return proc.performSync(request);
+        } else {
+            proc.performAsync(request);
+            return null;
+        }
+
+    }
+
+    @Override
+    public String toString() {
+        if (isDiscarded())
+            return "DISCARDED " + id.toString();
+        else if (isExcepted())
+            return id.toString() + " " + getResult();
+        else
+            return id.toString() + " " + statusOrException;
+    }
+
+    @Override
+    public void execute(AsyncReadGraph graph, T result) {
+        Collection<SessionTask> tasks = null;
+        synchronized(this) {
+            setResult(result);
+            setReady();
+            if(pendingTaskSupport != null)
+                tasks = pendingTaskSupport.executePending();
+        }
+        if(tasks != null)
+            for(SessionTask task : tasks)
+                ((ReadGraphImpl)graph).processor.scheduleNow(task);
+    }
+
+    @Override
+    public synchronized void exception(AsyncReadGraph graph, Throwable throwable) {
+        Collection<SessionTask> tasks = null;
+        synchronized(this) {
+            except(throwable);
+            if(pendingTaskSupport != null)
+                tasks = pendingTaskSupport.executePending();
+        }
+        if(tasks != null)
+            for(SessionTask task : tasks)
+                ((ReadGraphImpl)graph).processor.scheduleNow(task);
+    }
+
+    public void executeWhenResultIsAvailable(QueryProcessor processor, SessionTask task) {
+        boolean ready = false;
+        synchronized(this) {
+            if(pendingTaskSupport == null)
+                pendingTaskSupport = new PendingTaskSupport(this);
+            ready = pendingTaskSupport.executeWhenResultIsAvailable(task);
+        }
+        if(ready) {
+            processor.scheduleNow(task);
+        }
+    }
+
+    @Override
+    public String classId() {
+        return null;
+    }
+
+}