Uploaded image for project: 'camunda BPM'
  1. camunda BPM
  2. CAM-6203

Configure Tx Isolation level READ_COMMITTED in engine-IT on Wildfly

    Details

      Description

      • READ_COMMITTED is configured as default transaction isolation level on each datasource in wildfly

        Issue Links

          Activity

          Hide
          sebastian.menski Sebastian Menski added a comment - - edited

          Review hints:

          Show
          sebastian.menski Sebastian Menski added a comment - - edited Review hints: the test case TransactionIsolationLevelTest is missing on all branches except for master also please remove the println statement: https://github.com/camunda/camunda-bpm-platform-maintenance/blob/c88c284c1d90bf91d451185fb47e02768d3a3db6/qa/integration-tests-engine/src/test/java/org/camunda/bpm/integrationtest/functional/transactions/TransactionIsolationLevelTest.java#L43 why is the transaction level not set on h2? I think it should be consistent for all databases. can we please talk about merging branches without introducing a merge commits, that really isn't nice to review, for example there is now two different commits on master which are the same: 8fa5f7b1 vs 4fc104ad
          Show
          sebastian.menski Sebastian Menski added a comment - The Isolation Test is broken on 7.2: https://ci.camunda.com/jenkins/7.2/view/All/job/7.2-EE-engine-IT-jbosseap-62-postgresql-93/lastCompletedBuild/testReport/
          Show
          sebastian.menski Sebastian Menski added a comment - Isolation Test also broken on 7.5: https://ci.camunda.com/jenkins/7.5/job/7.5-engine-IT-wildfly-100-h2/lastCompletedBuild/testReport/org.camunda.bpm.integrationtest.functional.transactions/TransactionIsolationLevelTest/testTransactionIsolationLevelOnConnection/
          Show
          sebastian.menski Sebastian Menski added a comment - And 7.3 https://ci.camunda.com/jenkins/7.3/job/7.3-engine-IT-jboss-72-h2/lastCompletedBuild/testReport/org.camunda.bpm.integrationtest.functional.transactions/TransactionIsolationLevelTest/testTransactionIsolationLevelOnConnection/
          Show
          sebastian.menski Sebastian Menski added a comment - And 7.4: https://ci.camunda.com/jenkins/7.4/job/7.4-engine-IT-wildfly-82-h2/lastCompletedBuild/testReport/org.camunda.bpm.integrationtest.functional.transactions/TransactionIsolationLevelTest/testTransactionIsolationLevelOnConnection/

            People

            • Assignee:
              sebastian.menski Sebastian Menski
              Reporter:
              meyer Daniel Meyer
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development