You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tephra.apache.org by gokulavasan <gi...@git.apache.org> on 2017/03/01 23:51:18 UTC

[GitHub] incubator-tephra pull request #40: TEPHRA-227 Add a new command to get the s...

GitHub user gokulavasan opened a pull request:

    https://github.com/apache/incubator-tephra/pull/40

    TEPHRA-227 Add a new command to get the set of regions that have not been compacted

    Also adding a CLI command.
    
    JIRA : https://issues.apache.org/jira/browse/TEPHRA-227


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/gokulavasan/incubator-tephra fix/tephra-227

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-tephra/pull/40.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #40
    
----
commit c2b76408facb5d9b8169876c7830ec08759da455
Author: Gokul Gunasekaran <go...@cask.co>
Date:   2017-03-01T22:54:40Z

    (TEPHRA-227) Add new command to get the set of regions that have not been compacted

commit 7111ce76e19ed1be012e5a5d7f11b30e8dc757c2
Author: Gokul Gunasekaran <go...@cask.co>
Date:   2017-03-01T23:49:26Z

    Adding CLI command for the new getRegionsToBeCompacted method

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-tephra pull request #40: TEPHRA-227 Add a new command to get the s...

Posted by gokulavasan <gi...@git.apache.org>.
Github user gokulavasan closed the pull request at:

    https://github.com/apache/incubator-tephra/pull/40


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-tephra issue #40: TEPHRA-227 Add a new command to get the set of r...

Posted by poornachandra <gi...@git.apache.org>.
Github user poornachandra commented on the issue:

    https://github.com/apache/incubator-tephra/pull/40
  
    LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-tephra pull request #40: TEPHRA-227 Add a new command to get the s...

Posted by poornachandra <gi...@git.apache.org>.
Github user poornachandra commented on a diff in the pull request:

    https://github.com/apache/incubator-tephra/pull/40#discussion_r104040622
  
    --- Diff: tephra-hbase-compat-0.96/src/main/java/org/apache/tephra/hbase/txprune/InvalidListPruningDebug.java ---
    @@ -148,6 +232,9 @@ private void printUsage(PrintWriter pw) {
                      "provided as the limit, prune upper bounds of all regions are returned.");
         pw.println("prune-info region-name-as-string");
         pw.println("Desc: Prints out the Pruning information for the region 'region-name-as-string'");
    +    pw.println("to-compact-regions limit");
    +    pw.println("Desc: Prints out 'limit' number of regions that are active, but are not empty, " +
    +                 "but have not registered a prune upper bound.");
    --- End diff --
    
    `and have not registered a prune upper bound.`


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---