[jira] [Comment Edited] (TEPHRA-179) Tephra transaction manager breaks on zookeeper restart

2016-09-09 Thread Poorna Chandra (JIRA)
[ https://issues.apache.org/jira/browse/TEPHRA-179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15478772#comment-15478772 ] Poorna Chandra edited comment on TEPHRA-179 at 9/10/16 1:16 AM:

[jira] [Commented] (TEPHRA-179) Tephra transaction manager breaks on zookeeper restart

2016-09-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TEPHRA-179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15478777#comment-15478777 ] ASF GitHub Bot commented on TEPHRA-179: --- GitHub user poornachandra opened a pull req

[GitHub] incubator-tephra pull request #10: TEPHRA-179 Transaction service high avail...

2016-09-09 Thread poornachandra
GitHub user poornachandra opened a pull request: https://github.com/apache/incubator-tephra/pull/10 TEPHRA-179 Transaction service high availability changes Restructuring the Transaction Service classes to allow for HA restart while binding Transaction Manager and other classes as s

[jira] [Reopened] (TEPHRA-179) Tephra transaction manager breaks on zookeeper restart

2016-09-09 Thread Poorna Chandra (JIRA)
[ https://issues.apache.org/jira/browse/TEPHRA-179?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Poorna Chandra reopened TEPHRA-179: --- We found issues with HA while running integration tests > Tephra transaction manager breaks on zo

[GitHub] incubator-tephra pull request #9: Wait for Transaction Service to announce i...

2016-09-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-tephra/pull/9 --- 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

[GitHub] incubator-tephra pull request #9: Wait for Transaction Service to announce i...

2016-09-09 Thread poornachandra
Github user poornachandra commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/9#discussion_r78230869 --- Diff: tephra-core/src/test/java/org/apache/tephra/ThriftTransactionSystemTest.java --- @@ -88,10 +89,14 @@ protected void configure() {

[GitHub] incubator-tephra issue #9: Wait for Transaction Service to announce itself t...

2016-09-09 Thread anwar6953
Github user anwar6953 commented on the issue: https://github.com/apache/incubator-tephra/pull/9 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

[GitHub] incubator-tephra pull request #9: Wait for Transaction Service to announce i...

2016-09-09 Thread poornachandra
Github user poornachandra commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/9#discussion_r78230677 --- Diff: tephra-examples/pom.xml --- @@ -51,6 +55,53 @@ tephra-core ${project.version} + --- End dif

[GitHub] incubator-tephra pull request #9: Wait for Transaction Service to announce i...

2016-09-09 Thread anwar6953
Github user anwar6953 commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/9#discussion_r78227433 --- Diff: tephra-examples/pom.xml --- @@ -51,6 +55,53 @@ tephra-core ${project.version} + --- End diff --

[GitHub] incubator-tephra pull request #9: Wait for Transaction Service to announce i...

2016-09-09 Thread anwar6953
Github user anwar6953 commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/9#discussion_r78227091 --- Diff: tephra-core/src/test/java/org/apache/tephra/ThriftTransactionSystemTest.java --- @@ -88,10 +89,14 @@ protected void configure() {

[GitHub] incubator-tephra pull request #9: Wait for Transaction Service to announce i...

2016-09-09 Thread poornachandra
GitHub user poornachandra opened a pull request: https://github.com/apache/incubator-tephra/pull/9 Wait for Transaction Service to announce itself to fix flaky tests You can merge this pull request into a Git repository by running: $ git pull https://github.com/poornachandra/i

[GitHub] incubator-tephra pull request #8: Fix flaky test (PooledClientProviderTest)

2016-09-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-tephra/pull/8 --- 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

[GitHub] incubator-tephra issue #8: Fix flaky test (PooledClientProviderTest)

2016-09-09 Thread poornachandra
Github user poornachandra commented on the issue: https://github.com/apache/incubator-tephra/pull/8 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

[GitHub] incubator-tephra pull request #8: Fix flaky test (PooledClientProviderTest)

2016-09-09 Thread anwar6953
Github user anwar6953 commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/8#discussion_r78137565 --- Diff: tephra-core/src/test/java/org/apache/tephra/distributed/PooledClientProviderTest.java --- @@ -137,15 +137,15 @@ public Boolean call() thr

[GitHub] incubator-tephra pull request #8: Fix flaky test (PooledClientProviderTest)

2016-09-09 Thread poornachandra
Github user poornachandra commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/8#discussion_r78137526 --- Diff: tephra-core/src/test/java/org/apache/tephra/distributed/PooledClientProviderTest.java --- @@ -154,11 +154,10 @@ public Boolean call()

[GitHub] incubator-tephra pull request #8: Fix flaky test (PooledClientProviderTest)

2016-09-09 Thread poornachandra
Github user poornachandra commented on a diff in the pull request: https://github.com/apache/incubator-tephra/pull/8#discussion_r78137456 --- Diff: tephra-core/src/test/java/org/apache/tephra/distributed/PooledClientProviderTest.java --- @@ -137,15 +137,15 @@ public Boolean call()

[GitHub] incubator-tephra pull request #8: Fix flaky test (PooledClientProviderTest)

2016-09-09 Thread anwar6953
GitHub user anwar6953 opened a pull request: https://github.com/apache/incubator-tephra/pull/8 Fix flaky test (PooledClientProviderTest) Fix flaky test (PooledClientProviderTest). Previously, the entire client pool was not exhausted. You can merge this pull request into a Git repos