[GitHub] incubator-tephra pull request #34: TEPHRA-216 Handle empty transactional reg...

2017-02-13 Thread gokulavasan
Github user gokulavasan commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/34#discussion_r100868455 --- Diff: tephra-hbase-compat-0.96/src/test/java/org/apache/tephra/hbase/txprune/InvalidListPruneTest.java --- @@ -341,6 +310,87 @@ public

[GitHub] incubator-tephra pull request #34: TEPHRA-216 Handle empty transactional reg...

2017-02-10 Thread gokulavasan
Github user gokulavasan commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/34#discussion_r100655264 --- Diff: tephra-hbase-compat-1.1-base/src/main/java/org/apache/tephra/hbase/txprune/HBaseTransactionPruningPlugin.java --- @@ -322,6 +333,28

[GitHub] incubator-tephra pull request #34: TEPHRA-216 Handle empty transactional reg...

2017-02-10 Thread poornachandra
Github user poornachandra commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/34#discussion_r100648648 --- Diff: tephra-hbase-compat-1.1-base/src/main/java/org/apache/tephra/hbase/txprune/HBaseTransactionPruningPlugin.java --- @@ -322,6

[GitHub] incubator-tephra pull request #34: TEPHRA-216 Handle empty transactional reg...

2017-02-10 Thread poornachandra
Github user poornachandra commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/34#discussion_r100648684 --- Diff: tephra-hbase-compat-1.1-base/src/main/java/org/apache/tephra/hbase/txprune/DataJanitorState.java --- @@ -366,6 +371,56 @@ public

[GitHub] incubator-tephra pull request #34: TEPHRA-216 Handle empty transactional reg...

2017-02-10 Thread poornachandra
Github user poornachandra commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/34#discussion_r100647433 --- Diff: tephra-hbase-compat-1.1-base/src/main/java/org/apache/tephra/hbase/txprune/HBaseTransactionPruningPlugin.java --- @@ -288,23

[GitHub] incubator-tephra pull request #34: TEPHRA-216 Handle empty transactional reg...

2017-02-10 Thread gokulavasan
Github user gokulavasan commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/34#discussion_r100632563 --- Diff: tephra-hbase-compat-1.1-base/src/main/java/org/apache/tephra/hbase/txprune/HBaseTransactionPruningPlugin.java --- @@ -322,6 +333,28

[GitHub] incubator-tephra pull request #34: TEPHRA-216 Handle empty transactional reg...

2017-02-10 Thread gokulavasan
Github user gokulavasan commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/34#discussion_r100632110 --- Diff: tephra-hbase-compat-1.1-base/src/main/java/org/apache/tephra/hbase/txprune/HBaseTransactionPruningPlugin.java --- @@ -322,6 +333,28

[GitHub] incubator-tephra pull request #34: TEPHRA-216 Handle empty transactional reg...

2017-02-10 Thread anew
Github user anew commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/34#discussion_r100603640 --- Diff: tephra-hbase-compat-1.1-base/src/main/java/org/apache/tephra/hbase/txprune/HBaseTransactionPruningPlugin.java --- @@ -288,23 +288,27 @@

[GitHub] incubator-tephra pull request #34: TEPHRA-216 Handle empty transactional reg...

2017-02-10 Thread anew
Github user anew commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/34#discussion_r100608042 --- Diff: tephra-hbase-compat-1.1-base/src/main/java/org/apache/tephra/hbase/txprune/DataJanitorState.java --- @@ -366,6 +371,56 @@ public void

[GitHub] incubator-tephra pull request #34: TEPHRA-216 Handle empty transactional reg...

2017-02-10 Thread poornachandra
GitHub user poornachandra opened a pull request: https://github.com/apache/incubator-tephra/pull/34 TEPHRA-216 Handle empty transactional regions during inactive list pruning JIRA - https://issues.apache.org/jira/browse/TEPHRA-216 Approach: 1. Transaction Processor