Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-7484

Refactor SolrDispatchFilter to move all Solr specific implementation to another class

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 5.2
    • None
    • None

    Description

      Currently almost everything that's done in SDF.doFilter() is sequential. We should refactor it to clean up the code and make things easier to manage.

      Attachments

        1. SOLR-7484.patch
          0.9 kB
          Noble Paul
        2. SOLR-7484.patch
          65 kB
          Anshum Gupta
        3. SOLR-7484.patch
          65 kB
          Anshum Gupta
        4. SOLR-7484.patch
          68 kB
          Anshum Gupta
        5. SOLR-7484.patch
          64 kB
          Anshum Gupta
        6. SOLR-7484.patch
          63 kB
          Anshum Gupta
        7. SOLR-7484.patch
          63 kB
          Noble Paul
        8. SOLR-7484.patch
          63 kB
          Noble Paul
        9. SOLR-7484.patch
          26 kB
          Anshum Gupta
        10. SOLR-7484-override.patch
          2 kB
          Ryan McKinley

        Issue Links

        is depended upon by

        Sub-task - The sub-task of the issue SOLR-7275 Pluggable authorization module in Solr

        • Major - Major loss of function.
        • Closed
        is related to

        Improvement - An improvement or enhancement to an existing feature or task. SOLR-7623 Return the ability to override request dispatcher functions

        • Major - Major loss of function.
        • Closed
        relates to

        Test - A new unit, integration or system test. SOLR-7690 Add a test to assert that request dispatcher functions can be overridden

        • Minor - Minor loss of function, or other problem where easy workaround is present.
        • Open

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            anshum Anshum Gupta
            anshum Anshum Gupta
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment