CXF
  1. CXF
  2. CXF-3141

Move FIQL parser out to separate module

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.6
    • Component/s: JAX-RS
    • Labels:
      None
    • Estimated Complexity:
      Moderate

      Description

      The FIQL parser rocks. Not just because you guys did it, but because it's well documented. Bravo.

      My architecture has JAX-RS at a different tier than the persistence, and I need to pass the FIQL query deeper before I can parse it. To do that, I need to set a dependency on the entire cxf-rt-frontend-jaxrs jar and all it's dependencies. I'd rather not do that.

      Could it be broken out as it's own module? I can create the patch of it would be seriously considered.

      Thanks, Brian

        Activity

        Brian Topping created issue -
        Hide
        Sergey Beryozkin added a comment - - edited

        H
        Improving the modularity of the cxf-rt-frontend-jaxrs is on the map. Example, what can be pushed into separate modules and even subprojects. The search feature was meant to become the core part of the frontend to make it more useful for JAX-RS users.

        Perhaps we can consider moving the whole org.apache.cxf.jaxrs.ext.search package into a sep submodule, need to think more about it.

        cheers. Sergey

        Show
        Sergey Beryozkin added a comment - - edited H Improving the modularity of the cxf-rt-frontend-jaxrs is on the map. Example, what can be pushed into separate modules and even subprojects. The search feature was meant to become the core part of the frontend to make it more useful for JAX-RS users. Perhaps we can consider moving the whole org.apache.cxf.jaxrs.ext.search package into a sep submodule, need to think more about it. cheers. Sergey
        Mark Thomas made changes -
        Field Original Value New Value
        Workflow jira [ 12527380 ] Default workflow, editable Closed status [ 12605105 ]
        Hide
        Sergey Beryozkin added a comment -

        Moving the search extension into its own module is the best we can do right now. There will be still a dependency on the main jaxrs frontend because this module needs to be able to provide a custom Context implementation to be managed by the runtime. Having the actual parser code moved to yet another module is possible but I'm not sure where would it belong really...What can be done is to actually exclude the cxf-rt-frontend-jaxrs dependency but avoiding a cxf-api one is a bit unrealistic right now - please reopen this JIRA if you'd like me to pursue it...

        Show
        Sergey Beryozkin added a comment - Moving the search extension into its own module is the best we can do right now. There will be still a dependency on the main jaxrs frontend because this module needs to be able to provide a custom Context implementation to be managed by the runtime. Having the actual parser code moved to yet another module is possible but I'm not sure where would it belong really...What can be done is to actually exclude the cxf-rt-frontend-jaxrs dependency but avoiding a cxf-api one is a bit unrealistic right now - please reopen this JIRA if you'd like me to pursue it...
        Sergey Beryozkin made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Sergey Beryozkin [ sergey_beryozkin ]
        Fix Version/s 2.6 [ 12319252 ]
        Resolution Fixed [ 1 ]
        Sergey Beryozkin <sergeyb@apache.org> committed 960125550db5eddab32453eb1fdc6506044fdf93 (73 files)
        Reviews: none

        [CXF-4179,CXF-3141] Introducing ContextProviders and moving the search extension to its own module
        git-svn-id: https://svn.apache.org/repos/asf/cxf/trunk@1300208 13f79535-47bb-0310-9956-ffa450edef68

        cxf split-spring
        Sergey Beryozkin <sergeyb@apache.org> committed 8d2aebf1dab816148c78dea696c56838921d3bf7 (1 file)
        Daniel Kulp made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Sergey Beryozkin
            Reporter:
            Brian Topping
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 1h
              1h
              Remaining:
              Remaining Estimate - 1h
              1h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development