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

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

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Later
    • None
    • NeedMoreInfo
    • None
    • None
    • 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

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

              Dates

                Created:
                Updated:
                Resolved: