Derby
  1. Derby
  2. DERBY-5253

Engine code references org.apache.derby.shared.common.sanity.SanityManager

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 10.9.1.0
    • Fix Version/s: 10.8.2.2, 10.9.1.0
    • Component/s: Miscellaneous
    • Labels:
      None

      Description

      Derby engine code is importing org.apache.derby.shared.common.sanity.SanityManager, causing sealing violations. See DERBY-4669 why this fix is inadequate.
      Even though this won't fix the/all problems, using shared.common.sanity.SanityManager on the client and iapi.services.sanity.SanityManager in the engine code is a simple rule to follow. Violations are also easily addressed.

        Issue Links

          Activity

          Kristian Waagan created issue -
          Hide
          Kristian Waagan added a comment -

          Attached patch 1a, which replaces the imports from common with imports from iapi.

          Patch ready for review. I intend to commit this pretty soon.

          Show
          Kristian Waagan added a comment - Attached patch 1a, which replaces the imports from common with imports from iapi. Patch ready for review. I intend to commit this pretty soon.
          Kristian Waagan made changes -
          Field Original Value New Value
          Attachment derby-5253-1a-engine_imports.diff [ 12480866 ]
          Kristian Waagan made changes -
          Link This issue is related to DERBY-4669 [ DERBY-4669 ]
          Hide
          Kristian Waagan added a comment -

          Committed patch 1a to trunk with revision 1130077.
          Closing issue.

          Show
          Kristian Waagan added a comment - Committed patch 1a to trunk with revision 1130077. Closing issue.
          Kristian Waagan made changes -
          Status Open [ 1 ] Closed [ 6 ]
          Issue & fix info [Patch Available]
          Fix Version/s 10.9.0.0 [ 12316344 ]
          Resolution Fixed [ 1 ]
          Hide
          Kristian Waagan added a comment -

          Note the exception in iapi.services.info.JVMInfo. Here SanityManager is imported from shared.common to make it possible to keep iapi.services.sanity sealed. JVMInfo is included in both derby.jar and derbyclient.jar.

          Show
          Kristian Waagan added a comment - Note the exception in iapi.services.info.JVMInfo. Here SanityManager is imported from shared.common to make it possible to keep iapi.services.sanity sealed. JVMInfo is included in both derby.jar and derbyclient.jar.
          Hide
          Dag H. Wanvik added a comment -

          Would it be a good idea to make a small Wiki write-up on current state/ best practices when it comes to the use of shared code? It's not obvious..

          Show
          Dag H. Wanvik added a comment - Would it be a good idea to make a small Wiki write-up on current state/ best practices when it comes to the use of shared code? It's not obvious..
          Hide
          Kristian Waagan added a comment -

          Reopening for backport (needed for DERBY-4669).

          Show
          Kristian Waagan added a comment - Reopening for backport (needed for DERBY-4669 ).
          Kristian Waagan made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Hide
          Kristian Waagan added a comment -

          Backported to 10.8 with revision 1136020.
          Closing again.

          Show
          Kristian Waagan added a comment - Backported to 10.8 with revision 1136020. Closing again.
          Kristian Waagan made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Fix Version/s 10.8.1.4 [ 12316500 ]
          Resolution Fixed [ 1 ]
          Hide
          Kristian Waagan added a comment -

          Dag,

          Yes, a wiki could be a useful thing - or maybe freshen up the existing package.html we have.
          Before doing anything about this, I'd have to do some research though...

          Show
          Kristian Waagan added a comment - Dag, Yes, a wiki could be a useful thing - or maybe freshen up the existing package.html we have. Before doing anything about this, I'd have to do some research though...
          Knut Anders Hatlen made changes -
          Fix Version/s 10.8.1.5 [ 12316676 ]
          Fix Version/s 10.8.1.4 [ 12316500 ]
          Myrna van Lunteren made changes -
          Fix Version/s 10.8.2.0 [ 12317955 ]
          Fix Version/s 10.8.1.6 [ 12316676 ]
          Myrna van Lunteren made changes -
          Fix Version/s 10.8.2.2 [ 12317968 ]
          Fix Version/s 10.8.2.0 [ 12317955 ]
          Gavin made changes -
          Workflow jira [ 12615046 ] Default workflow, editable Closed status [ 12801065 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Closed Closed
          1d 14h 54m 1 Kristian Waagan 01/Jun/11 11:02
          Closed Closed Reopened Reopened
          14d 3h 16m 1 Kristian Waagan 15/Jun/11 14:19
          Reopened Reopened Closed Closed
          39s 1 Kristian Waagan 15/Jun/11 14:19

            People

            • Assignee:
              Kristian Waagan
              Reporter:
              Kristian Waagan
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development