#534 some changes to ensure the transaction mode is right.

MarkLogic 8.0-7 has a bug regarding transaction mode, which is fixed in
8.0-7.1. Made some changes to get the extension to work right in 8.0-7.

Author: dmcassel

marklogic-data-hub/src/main/resources/ml-modules/root/com.marklogic.hub/lib/job-lib.xqy

@@ -22,6 +22,8 @@ declare function job:delete-jobs-and-traces($job-ids as xs:string*)
         job:delete-traces($job-ids)
       },
       <options xmlns="xdmp:eval">
+        <update>true</update>
+        <isolation>different-transaction</isolation>
         <database>{xdmp:database($config:TRACE-DATABASE)}</database>
       </options>
     )

marklogic-data-hub/src/main/resources/ml-modules/services/delete-jobs.xqy

@@ -27,15 +27,28 @@ declare namespace rapi = "http://marklogic.com/rest-api";
 
 declare option xdmp:mapping "false";
 
+(:
+ : REST API extension to delete jobs and associated traces.
+ : Note: it shouldn't be necessary to use xdmp:invoke-function below, but a bug
+ : in ML 8.0-7 (fixed in 8.0-7.1) messes up the transaction mode.
+ :)
 declare %rapi:transaction-mode("update") function service:post(
     $context as map:map,
     $params  as map:map,
     $input   as document-node()*
-) as document-node()*
+) as document-node()?
 {
   perf:log('/v1/resources/delete-jobs:post', function() {
-    let $job-ids := fn:tokenize(map:get($params, "jobIds"), ",")
-    return
-      job:delete-jobs-and-traces($job-ids)
+    xdmp:invoke-function(
+      function() {
+        let $job-ids := fn:tokenize(map:get($params, "jobIds"), ",")
+        return
+          job:delete-jobs-and-traces($job-ids)
+      },
+      <options xmlns="xdmp:eval">
+        <update>true</update>
+        <isolation>same-statement</isolation>
+      </options>
+    )
   })
 };