Uploaded image for project: 'CXF'
  1. CXF
  2. CXF-5970

review org.apache.cxf.jaxrs.provider.ProviderFactory.Message*ReaderComparator

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Later
    • Affects Version/s: None
    • Fix Version/s: NeedMoreInfo
    • Component/s: None
    • Labels:
      None
    • Estimated Complexity:
      Unknown

      Description

      Actually the task can be rephrase saying rework org.apache.cxf.jaxrs.provider.ProviderFactory#*Comparators

      Current implementations don't seem to have an absolute order which means sorting is not as deterministic as it should be under java >= 7.

      Other point is as a user you set a List of providers (you probably already sorted if important) so you want to ensure your order is respected at runtime (ie org.apache.cxf.jaxrs.provider.ProviderFactory#message[Readers|Writers] respect it).

      If you think this sorting is important maybe introducing in org.apache.cxf.jaxrs.AbstractJAXRSFactoryBean and org.apache.cxf.jaxrs.provider.ProviderFactory a boolean sortProviders can be enough to prevent calling sortXXX().

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                sergey_beryozkin Sergey Beryozkin
                Reporter:
                romain.manni-bucau Romain Manni-Bucau
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: