Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Fix Version/s: 0.4
    • Component/s: Core
    • Labels:
      None

      Description

      everything that is per-node should be moved into JMX.

      everything that is not, should be moved into a new cluster management tool (may be web based eventually; command line is probably simplest just to get started with).

        Activity

        Hide
        Jonathan Ellis added a comment -

        I kinda think we should leave out the serialversionuid, because that says We Care About Deserializing Old Versions Of The Class when in this case we do not. (It also says "I will remember to update this number every time I change a field in this class" which I don't want to commit to unnecessarily.)

        Other than that, +1.

        Show
        Jonathan Ellis added a comment - I kinda think we should leave out the serialversionuid, because that says We Care About Deserializing Old Versions Of The Class when in this case we do not. (It also says "I will remember to update this number every time I change a field in this class" which I don't want to commit to unnecessarily.) Other than that, +1.
        Hide
        Eric Evans added a comment -

        That makes sense. The attachment with the most recent timestamp does away with the serial.

        Show
        Eric Evans added a comment - That makes sense. The attachment with the most recent timestamp does away with the serial.
        Hide
        Jonathan Ellis added a comment -

        applied most recent

        Show
        Jonathan Ellis added a comment - applied most recent
        Hide
        Hudson added a comment -

        Integrated in Cassandra #100 (See http://hudson.zones.apache.org/hudson/job/Cassandra/100/)
        add getRangeToEndPointMap to JMX. Requires making Token and Range Serializable. Patch by Eric Evans; reviewed by jbellis for

        Show
        Hudson added a comment - Integrated in Cassandra #100 (See http://hudson.zones.apache.org/hudson/job/Cassandra/100/ ) add getRangeToEndPointMap to JMX. Requires making Token and Range Serializable. Patch by Eric Evans; reviewed by jbellis for
        Hide
        Eric Evans added a comment -

        0001-CASSANDRA-211-jmx-load-info-and-generation-number.txt

        This one adds instrumentation for load info and the generation number (currently viewable from the cluster "tab" in the webui).

        Show
        Eric Evans added a comment - 0001- CASSANDRA-211 -jmx-load-info-and-generation-number.txt This one adds instrumentation for load info and the generation number (currently viewable from the cluster "tab" in the webui).
        Hide
        Eric Evans added a comment -

        0001-CASSANDRA-211-rename-Table.doGC-to-forceCleanup.txt
        0002-CASSANDRA-211-expose-compaction-via-JMX.txt

        0001 changes a badly chosen method name, 0002 exposes an operation via JMX for the compaction of all tables.

        Show
        Eric Evans added a comment - 0001- CASSANDRA-211 -rename-Table.doGC-to-forceCleanup.txt 0002- CASSANDRA-211 -expose-compaction-via-JMX.txt 0001 changes a badly chosen method name, 0002 exposes an operation via JMX for the compaction of all tables.
        Hide
        Jonathan Ellis added a comment -

        committed load info, rename, and compaction patches

        Show
        Jonathan Ellis added a comment - committed load info, rename, and compaction patches
        Hide
        Eric Evans added a comment -

        This still needs a way of addressing the load/bootstrap functionality (despite that not actually working the webui), but I could do with some feedback. Comments welcome.

        Show
        Eric Evans added a comment - This still needs a way of addressing the load/bootstrap functionality (despite that not actually working the webui), but I could do with some feedback. Comments welcome.
        Hide
        Hudson added a comment -

        Integrated in Cassandra #103 (See http://hudson.zones.apache.org/hudson/job/Cassandra/103/)
        expose compaction via JMX. patch by Eric Evans; reviewed by jbellis for
        rename Table.doGC to forceCleanup. patch by Eric Evans; reviewed by jbellis for
        To make available the Cassandra specific information available on the
        node detail page:

        • Exposed the existing StorageService.getLoadInfo()
        • Created StorageService.getCurrentGenerationNumber() and added to
          MBean interface.

        patch by Eric Evans; reviewed by jbellis for

        Show
        Hudson added a comment - Integrated in Cassandra #103 (See http://hudson.zones.apache.org/hudson/job/Cassandra/103/ ) expose compaction via JMX. patch by Eric Evans; reviewed by jbellis for rename Table.doGC to forceCleanup. patch by Eric Evans; reviewed by jbellis for To make available the Cassandra specific information available on the node detail page: Exposed the existing StorageService.getLoadInfo() Created StorageService.getCurrentGenerationNumber() and added to MBean interface. patch by Eric Evans; reviewed by jbellis for
        Hide
        Jonathan Ellis added a comment -

        committed nodeprobe.

        Show
        Jonathan Ellis added a comment - committed nodeprobe.
        Hide
        Hudson added a comment -

        Integrated in Cassandra #111 (See http://hudson.zones.apache.org/hudson/job/Cassandra/111/)
        add nodeprobe cli tool. patch by Eric Evans; reviewed by jbellis for

        Show
        Hudson added a comment - Integrated in Cassandra #111 (See http://hudson.zones.apache.org/hudson/job/Cassandra/111/ ) add nodeprobe cli tool. patch by Eric Evans; reviewed by jbellis for
        Hide
        Jonathan Ellis added a comment -

        the only thing left in my mind is "remove the web ui." any reason not to proceed with that?

        Show
        Jonathan Ellis added a comment - the only thing left in my mind is "remove the web ui." any reason not to proceed with that?
        Hide
        Jonathan Ellis added a comment -

        (added a possible enhancement to nodeprobe to a new ticket in CASSANDRA-252 – note that this is functionality above and beyond what the web ui did)

        Show
        Jonathan Ellis added a comment - (added a possible enhancement to nodeprobe to a new ticket in CASSANDRA-252 – note that this is functionality above and beyond what the web ui did)
        Hide
        Eric Evans added a comment -

        The only thing that you can currently do with the web interface, that you cannot do otherwise, is perform storage operations against the local node. Since it is a local-only operation, I can see where that might be useful for testing and verification, but anecdotal evidence seems to suggest that when people do use it, they simple use it to toy around while learning, or while demonstrating the data model (and that can be accomplished with the CLI).

        I vote to remove it, but you might want to give a little extra time/opportunity for others to object. Maybe a post to -user and/or -dev?

        Show
        Eric Evans added a comment - The only thing that you can currently do with the web interface, that you cannot do otherwise, is perform storage operations against the local node. Since it is a local-only operation, I can see where that might be useful for testing and verification, but anecdotal evidence seems to suggest that when people do use it, they simple use it to toy around while learning, or while demonstrating the data model (and that can be accomplished with the CLI). I vote to remove it, but you might want to give a little extra time/opportunity for others to object. Maybe a post to -user and/or -dev?
        Hide
        Jonathan Ellis added a comment -

        patch to r/m web interface.

        I don't think I want to bikeshed this one.

        Show
        Jonathan Ellis added a comment - patch to r/m web interface. I don't think I want to bikeshed this one.
        Hide
        Eric Evans added a comment -

        Your patch looks good, but I found a few more things to cleanup, and we should probably stab a note in the changelog.

        Patch attached.

        Show
        Eric Evans added a comment - Your patch looks good, but I found a few more things to cleanup, and we should probably stab a note in the changelog. Patch attached.
        Hide
        Jonathan Ellis added a comment -

        excellent. committed.

        Show
        Jonathan Ellis added a comment - excellent. committed.
        Hide
        Hudson added a comment -

        Integrated in Cassandra #121 (See http://hudson.zones.apache.org/hudson/job/Cassandra/121/)
        r/m http server (use cli, nodeprobe, jmx as necessary instead).
        patch by jbellis and Eric Evans for

        Show
        Hudson added a comment - Integrated in Cassandra #121 (See http://hudson.zones.apache.org/hudson/job/Cassandra/121/ ) r/m http server (use cli, nodeprobe, jmx as necessary instead). patch by jbellis and Eric Evans for

          People

          • Assignee:
            Eric Evans
            Reporter:
            Jonathan Ellis
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development