[GitHub] incubator-tephra pull request #61: TEPHRA-263 Enforce TTL, regardless of any...

2017-09-18 Thread poornachandra
Github user poornachandra commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/61#discussion_r139523472 --- Diff: tephra-hbase-compat-1.0/src/main/java/org/apache/tephra/hbase/coprocessor/TransactionVisibilityFilter.java --- @@ -89,13 +89,13 @@

[GitHub] incubator-tephra pull request #61: TEPHRA-263 Enforce TTL, regardless of any...

2017-09-18 Thread anwar6953
Github user anwar6953 commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/61#discussion_r139515752 --- Diff: tephra-core/src/main/java/org/apache/tephra/util/TxUtils.java --- @@ -75,12 +75,9 @@ public static long

[GitHub] incubator-tephra pull request #61: TEPHRA-263 Enforce TTL, regardless of any...

2017-09-18 Thread anwar6953
Github user anwar6953 commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/61#discussion_r139512591 --- Diff: tephra-hbase-compat-1.0/src/test/java/org/apache/tephra/hbase/coprocessor/TransactionVisibilityFilterTest.java --- @@ -290,44 +290,52

[GitHub] incubator-tephra pull request #61: TEPHRA-263 Enforce TTL, regardless of any...

2017-09-17 Thread chtyim
Github user chtyim commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/61#discussion_r139303849 --- Diff: tephra-core/src/main/java/org/apache/tephra/util/TxUtils.java --- @@ -75,12 +75,9 @@ public static long

[GitHub] incubator-tephra pull request #61: TEPHRA-263 Enforce TTL, regardless of any...

2017-09-16 Thread anew
Github user anew commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/61#discussion_r139287848 --- Diff: tephra-hbase-compat-1.0/src/test/java/org/apache/tephra/hbase/coprocessor/TransactionVisibilityFilterTest.java --- @@ -290,44 +290,52 @@

[GitHub] incubator-tephra pull request #61: TEPHRA-263 Enforce TTL, regardless of any...

2017-09-16 Thread anew
Github user anew commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/61#discussion_r139287786 --- Diff: tephra-core/src/main/java/org/apache/tephra/util/TxUtils.java --- @@ -75,12 +75,9 @@ public static long getOldestVisibleTimestamp(Map

[GitHub] incubator-tephra pull request #61: TEPHRA-263 Enforce TTL, regardless of any...

2017-09-16 Thread anew
Github user anew commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/61#discussion_r139287927 --- Diff: tephra-hbase-compat-1.0/src/test/java/org/apache/tephra/hbase/coprocessor/TransactionVisibilityFilterTest.java --- @@ -290,44 +290,52 @@

[GitHub] incubator-tephra pull request #61: TEPHRA-263 Enforce TTL, regardless of any...

2017-09-15 Thread anwar6953
GitHub user anwar6953 opened a pull request: https://github.com/apache/incubator-tephra/pull/61 TEPHRA-263 Enforce TTL, regardless of any in-progress transactions. https://issues.apache.org/jira/browse/TEPHRA-263 I'll apply the diff to the other hbase compat modules once these