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

      Hide
      dsmiley David Smiley added a comment -

      +1!

      Show
      dsmiley David Smiley added a comment - +1!
      Hide
      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
      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
      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
      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
      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