Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor 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
        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
        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
        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
        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
        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
        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 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 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
        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
        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
        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
        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
        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
        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 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 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 Soztutar added a comment -

        Closing this issue after 0.99.0 release.

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

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development