0

我有一个小程序,它应该一个一个地读取 SQL 查询/命令并针对数据库执行它们。

如果查询成功执行,则执行下一个查询。如果执行一个查询时出错,程序应该一起停止执行。

我有代码,只是即使出现异常,查询仍会继续执行。

(defn main
   []
   (loop [queries (get-all-queries)
          querycount 1]
     (let [q (first queries)]
        (println (format "currently processing query %s", querycount))
        (cond (nil? q) (println "All Queries ran successfully.")
              :else (do
                      (cond (= (:status (process-query q querycount)) "OK") 
                               (recur (rest queries) (+querycount 1)))
                      :else (println "An error occured while running queries")))))))


 (defn process-query
     [query query-count]
     (let [{query-body :query-body, is-query-running? :is-query-running?} query
           my-agent (agent 
                       {:error false, :query-count query-count} 
                       :error-handler handler-fn)]
        (send my-agent (fn[_]
                          (execute-query! db query-body)))))
        (loop [is-query-running? (is-query-running?)
               error? (:error @my-agent)]
           (cond error? (do (println "Error") 
                            {:status "ERROR" :error-msg (:error-msg @my-agent)})
           (and (not is-query-running?) (not error?)) (do (println "Success") 
                                                          {:status "OK"})
           (:else (do
                    (Thread/sleep 2000)
                    (recur (is-query-running?) (:error @my-agent)))))))


(defn handler-fn
  [agent exception]
  (println (format "an exception occured : %s" exception))
  (if (instance? java.sql.BatchUpdateException exception)
      (println (.getNextException exception)))
  (send agent (? [_] {:error true, :error-message exception}))
  (throw exception))

我使用代理的原因是我有一些查询需要 4 小时才能运行。当这种情况发生时,数据库不会通知程序查询已经完成。相反,程序卡住了。所以,相反,我经常轮询以检查查询是否已经完成。

  • 这是完成我想做的事情的最佳方式吗?
  • 我应该使用任何其他并发原语吗?
  • 我什至需要并发原语吗?
  • 我已经想了很久了。
4

2 回答 2

1

主要问题似乎是:一方面,您写道长查询永远不会返回,即它们甚至不会抛出异常。另一方面,代理的错误检测机制基于捕获异常。

我认为您需要做的不是(主要)检查是否捕获了异常,而是在返回 falseexecute-query时是否实际上返回了有效结果。is-query-running?

关于正确的并发原语,我建议使用未来而不是代理。它们比代理更简单,因为它们只能返回一个值,而不是多次更改其状态,并且它们的错误处理方式是简单地返回异常而不是常规返回值。

然后,您可以遵循这个实现思路:在循环中,deref在将来执行一个 with timeout 。如果 的返回值derefexecute-query!定期返回的值,则返回"OK"(分别在future正文中添加第二个表达式作为清晰可识别的返回值,例如关键字:ok)。否则,如果 的返回值deref是异常,"ERROR":error-msg像现在一样从异常中返回。最后,如果返回值是你给deref, 调用的超时值is-query-running?。如果为真,则再循环一次,如果为假,则返回ERROR一个特殊的值:error-msg,它表示您的查询已结束,既不返回也不抛出异常。(并且可能打电话给future-cancel所以你不会泄漏永无止境的execute-query!调用线程。)

于 2013-11-12T15:14:03.833 回答
1

我认为你需要使用 core.async 来解决这种工作流看看http://clojure.com/blog/2013/06/28/clojure-core-async-channels.html
这个库会让你用所涉及的相关异步任务检查您的条件

一些可以帮助您的资源
http://www.infoq.com/news/2013/07/core-async
https://www.youtube.com/watch?v=AhxcGGeh5ho

于 2013-11-11T11:58:30.530 回答