Uploaded image for project: 'Isis'
  1. Isis
  2. ISIS-1453

SessionManagementService#nextSession broken (in 1.13.0-SNAPSHOT)

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.13.0
    • Component/s: Core
    • Labels:
      None

      Description

      Calls the incorrect API to start new transaction, resulting in Isis attempting to start the JDO xactn twice.

        Activity

        Hide
        jira-bot ASF subversion and git services added a comment -

        Commit 3b7a74c551b1ff7bbd886b7fa22533df8d6d9a3c in isis's branch refs/heads/master from Dan Haywood
        [ https://git-wip-us.apache.org/repos/asf?p=isis.git;h=3b7a74c ]

        ISIS-1453: fixes SessionManagementService to call the correct API to set up the next xactn.

        Use of the wrong API was resulting in the Isis framework's notion of the current transaction being out-of-whack with JDO's, resulting in the framework attempting to start the same JDO xactn twice.

        In addition, fixed handling of closing the command when first session is closed; was resulting in NPE because the interaction.Execution is still current and thus integration.getPriorExecution() returns null (resulting in an NPE).

        Show
        jira-bot ASF subversion and git services added a comment - Commit 3b7a74c551b1ff7bbd886b7fa22533df8d6d9a3c in isis's branch refs/heads/master from Dan Haywood [ https://git-wip-us.apache.org/repos/asf?p=isis.git;h=3b7a74c ] ISIS-1453 : fixes SessionManagementService to call the correct API to set up the next xactn. Use of the wrong API was resulting in the Isis framework's notion of the current transaction being out-of-whack with JDO's, resulting in the framework attempting to start the same JDO xactn twice. In addition, fixed handling of closing the command when first session is closed; was resulting in NPE because the interaction.Execution is still current and thus integration.getPriorExecution() returns null (resulting in an NPE).

          People

          • Assignee:
            danhaywood Dan Haywood
            Reporter:
            danhaywood Dan Haywood
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development