Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0-pr
    • Labels:
      None

      Description

      If a OSGi bundle includes both api+impl without META-INF/services entry, the default impl won't be loaded by TCC(since TCC may be webapp classloader)

        Activity

        jitu created issue -
        jitu made changes -
        Field Original Value New Value
        Fix Version/s 1.0-pr [ 16133 ]
        Hide
        jitu added a comment -

        Not specifying how a default provider is loaded.

        • Also considered specifying a system property for a provider name. The order considered was:
          1) system property
          2) ServiceLoader
          3) default provider

        That doesn't work well when a webapp bundles its own provider in a war file (if a system property is already defined).

        • If we define the order to be
          1) ServiceLoader
          2) System property
          3) default provider

        then system property doesn't work well(since one of the providers are picked up using ServiceLoader).

        so not specifying system property as it is of limited use.

        Show
        jitu added a comment - Not specifying how a default provider is loaded. Also considered specifying a system property for a provider name. The order considered was: 1) system property 2) ServiceLoader 3) default provider That doesn't work well when a webapp bundles its own provider in a war file (if a system property is already defined). If we define the order to be 1) ServiceLoader 2) System property 3) default provider then system property doesn't work well(since one of the providers are picked up using ServiceLoader). so not specifying system property as it is of limited use.
        jitu made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            jitu
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: