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

Refactor OverseerMessageHandler and make smaller classes

        Details

        • Type: Task
        • Status: Closed
        • Priority: Trivial
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 6.2
        • Component/s: None
        • Security Level: Public (Default Security Level. Issues are Public)

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          dsmiley David Smiley added a comment -

          +1!

          Show
          dsmiley David Smiley added a comment - +1!
          Hide
          Permalink
          jira-bot ASF subversion and git services added a comment -

          Commit 292644f9a5f961955ce56862dcaf29e7debc4c48 in lucene-solr's branch refs/heads/branch_6x from Noble Paul
          [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=292644f ]

          SOLR-9421: Refactored out OverseerCollectionMessageHandler to smaller classes

          Show
          jira-bot ASF subversion and git services added a comment - Commit 292644f9a5f961955ce56862dcaf29e7debc4c48 in lucene-solr's branch refs/heads/branch_6x from Noble Paul [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=292644f ] SOLR-9421 : Refactored out OverseerCollectionMessageHandler to smaller classes
          Hide
          Permalink
          jira-bot ASF subversion and git services added a comment -

          Commit bbd1efe5d8f547e0503d2a39abbfd3849019c77f in lucene-solr's branch refs/heads/master from Noble Paul
          [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=bbd1efe ]

          SOLR-9421: Refactored out OverseerCollectionMessageHandler to smaller classes

          Show
          jira-bot ASF subversion and git services added a comment - Commit bbd1efe5d8f547e0503d2a39abbfd3849019c77f in lucene-solr's branch refs/heads/master from Noble Paul [ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=bbd1efe ] SOLR-9421 : Refactored out OverseerCollectionMessageHandler to smaller classes
          Hide
          Permalink
          dsmiley David Smiley added a comment -

          Noble Paul can you add a package for all these classes? It would help organize the code around here better. Perhaps Overseer*.java should go there too (+ classes only used by them, like...?) and we call this package "overseer"? If that's too much then maybe just call the package "overseer-cmd".

          Show
          dsmiley David Smiley added a comment - Noble Paul can you add a package for all these classes? It would help organize the code around here better. Perhaps Overseer*.java should go there too (+ classes only used by them, like...?) and we call this package "overseer"? If that's too much then maybe just call the package "overseer-cmd".
          Hide
          Permalink
          noble.paul Noble Paul added a comment -

          I think we should open another ticket for that. I'm glad I have split this behemoth into pieces so that I can add features w/o getting bogged down

          Show
          noble.paul Noble Paul added a comment - I think we should open another ticket for that. I'm glad I have split this behemoth into pieces so that I can add features w/o getting bogged down
          Hide
          Permalink
          mikemccand Michael McCandless added a comment -

          Bulk close resolved issues after 6.2.0 release.

          Show
          mikemccand Michael McCandless added a comment - Bulk close resolved issues after 6.2.0 release.

            People

            • Assignee:
              noble.paul Noble Paul
              Reporter:
              noble.paul Noble Paul
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development