Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Invalid
    • Affects Version/s: None
    • Fix Version/s: 0.99.0
    • Component/s: documentation, util
    • Labels:
      None

      Description

      RegionSplitter is a utility for partitioning a table based on some split algorithm. Those same algorithms are exposed via the shell create command. There's no value in having two ways to access the same functionality. Ensure the main method doesn't provide any functionality absent from the shell and remove it.

        Activity

        Hide
        jmspaggi Jean-Marc Spaggiari added a comment -

        Is there any real value to remove it? There is some useful methods like rollingSplit in it. And I don't think it's causing any issue to keep it, no?

        Show
        jmspaggi Jean-Marc Spaggiari added a comment - Is there any real value to remove it? There is some useful methods like rollingSplit in it. And I don't think it's causing any issue to keep it, no?
        Hide
        ndimiduk Nick Dimiduk added a comment -

        Sorry, I corrected the title after creating the issue. My intent is to remove the main() method, not the class itself. These useful methods and the split policy definitions are used by the shell, so let's keep those.

        Show
        ndimiduk Nick Dimiduk added a comment - Sorry, I corrected the title after creating the issue. My intent is to remove the main() method, not the class itself. These useful methods and the split policy definitions are used by the shell, so let's keep those.
        Hide
        jmspaggi Jean-Marc Spaggiari added a comment -

        If you remove the main, can you please make sure it's documented somewhere how to achieve the same thing with the shell, in case someone is still using it?

        Show
        jmspaggi Jean-Marc Spaggiari added a comment - If you remove the main, can you please make sure it's documented somewhere how to achieve the same thing with the shell, in case someone is still using it?
        Hide
        anoop.hbase Anoop Sam John added a comment -

        Yes Shell uses this.
        Ideally Shell should work with out hbase server side jars This class is in hbase-server package.

        Show
        anoop.hbase Anoop Sam John added a comment - Yes Shell uses this. Ideally Shell should work with out hbase server side jars This class is in hbase-server package.
        Hide
        phobos182 Jeremy Carroll added a comment -

        FWIW we currently use the org.apache.hadoop.hbase.util.RegionSplitter with online rolling splits to perform capacity planning on live clusters. The HBase shell only allows for Splits for newly created tables I believe.

        Show
        phobos182 Jeremy Carroll added a comment - FWIW we currently use the org.apache.hadoop.hbase.util.RegionSplitter with online rolling splits to perform capacity planning on live clusters. The HBase shell only allows for Splits for newly created tables I believe.
        Hide
        ndimiduk Nick Dimiduk added a comment -

        Jeremy Carroll fascinating. Would you be willing to contribute a section on the tool's use to the community book? You bring the content, I'll juggle the xml nonsense; deal?

        Resolving as invalid.

        Show
        ndimiduk Nick Dimiduk added a comment - Jeremy Carroll fascinating. Would you be willing to contribute a section on the tool's use to the community book ? You bring the content, I'll juggle the xml nonsense; deal? Resolving as invalid.
        Hide
        phobos182 Jeremy Carroll added a comment -

        Very late reply from myself. I can document the procedures we use internally. Where would I put this information?

        Show
        phobos182 Jeremy Carroll added a comment - Very late reply from myself. I can document the procedures we use internally. Where would I put this information?
        Hide
        stack stack added a comment -

        Jeremy Carroll Stick a few lines here and if you are nice our mighty Misty Stanley-Jones will make a poem of them and stick it in refguide.

        Show
        stack stack added a comment - Jeremy Carroll Stick a few lines here and if you are nice our mighty Misty Stanley-Jones will make a poem of them and stick it in refguide.
        Hide
        enis Enis Soztutar added a comment -

        Closing this issue after 0.99.0 release.

        Show
        enis Enis Soztutar added a comment - Closing this issue after 0.99.0 release.

          People

          • Assignee:
            Unassigned
            Reporter:
            ndimiduk Nick Dimiduk
          • Votes:
            0 Vote for this issue
            Watchers:
            7 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development