[GitHub] mikewalch commented on a change in pull request #953: fixes #917 Added read locks

2017-10-31 Thread GitBox
mikewalch commented on a change in pull request #953: fixes #917 Added read locks URL: https://github.com/apache/fluo/pull/953#discussion_r148025151 ## File path: modules/api/src/main/java/org/apache/fluo/api/client/TransactionBase.java ## @@ -60,4 +60,26 @@ * encoded

[GitHub] mikewalch commented on a change in pull request #953: fixes #917 Added read locks

2017-10-26 Thread GitBox
mikewalch commented on a change in pull request #953: fixes #917 Added read locks URL: https://github.com/apache/fluo/pull/953#discussion_r147271399 ## File path: modules/api/src/main/java/org/apache/fluo/api/client/TransactionBase.java ## @@ -60,4 +60,26 @@ * encoded

[GitHub] mikewalch commented on a change in pull request #953: fixes #917 Added read locks

2017-10-25 Thread GitBox
mikewalch commented on a change in pull request #953: fixes #917 Added read locks URL: https://github.com/apache/fluo/pull/953#discussion_r146931819 ## File path: modules/accumulo/src/main/java/org/apache/fluo/accumulo/util/ColumnConstants.java ## @@ -26,6 +26,7 @@ pub

[GitHub] mikewalch commented on a change in pull request #953: fixes #917 Added read locks

2017-10-25 Thread GitBox
mikewalch commented on a change in pull request #953: fixes #917 Added read locks URL: https://github.com/apache/fluo/pull/953#discussion_r146949606 ## File path: modules/integration/src/test/java/org/apache/fluo/integration/impl/ReadLockIT.java ## @@ -0,0 +1,548 @@ +/* +

[GitHub] mikewalch commented on a change in pull request #953: fixes #917 Added read locks

2017-10-25 Thread GitBox
mikewalch commented on a change in pull request #953: fixes #917 Added read locks URL: https://github.com/apache/fluo/pull/953#discussion_r146957600 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/LockResolver.java ## @@ -52,38 +61,61 @@ public class

[GitHub] mikewalch commented on a change in pull request #953: fixes #917 Added read locks

2017-10-25 Thread GitBox
mikewalch commented on a change in pull request #953: fixes #917 Added read locks URL: https://github.com/apache/fluo/pull/953#discussion_r146955064 ## File path: modules/api/src/main/java/org/apache/fluo/api/client/TransactionBase.java ## @@ -60,4 +60,26 @@ * encoded

[GitHub] mikewalch commented on a change in pull request #953: fixes #917 Added read locks

2017-10-25 Thread GitBox
mikewalch commented on a change in pull request #953: fixes #917 Added read locks URL: https://github.com/apache/fluo/pull/953#discussion_r146953234 ## File path: modules/integration/src/test/java/org/apache/fluo/integration/impl/ReadLockIT.java ## @@ -0,0 +1,548 @@ +/* +

[GitHub] mikewalch commented on a change in pull request #953: fixes #917 Added read locks

2017-10-23 Thread GitBox
mikewalch commented on a change in pull request #953: fixes #917 Added read locks URL: https://github.com/apache/fluo/pull/953#discussion_r146357935 ## File path: modules/integration/src/test/resources/log4j.properties ## @@ -36,3 +36,5 @@ log4j.logger.org.apache.zookeeper