[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_r146931819 ## File path: modules/accumulo/src/main/java/org/apache/fluo/accumulo/util/ColumnConstants.java ## @@ -26,6 +26,7 @@

[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

[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 @@ *

[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 @@