[GitHub] incubator-fluo-website issue #55: Add mjwall as contributor

2017-05-10 Thread mjwall
Github user mjwall commented on the issue: https://github.com/apache/incubator-fluo-website/pull/55 thanks @keith-turner --- 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

[GitHub] incubator-fluo issue #832: closes #809 - cleanup buffered data

2017-05-10 Thread mjwall
Github user mjwall commented on the issue: https://github.com/apache/incubator-fluo/pull/832 thanks @keith-turner, I'll make a PR like you suggested for the contributors page --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-fluo issue #827: Add copyTo() method to bytes

2017-05-09 Thread mjwall
Github user mjwall commented on the issue: https://github.com/apache/incubator-fluo/issues/827 I can take a swing at this one. Is there a way for us to assign issues in GH? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-fluo pull request #832: closes #809 - cleanup buffered data

2017-05-09 Thread mjwall
Github user mjwall commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/832#discussion_r115620114 --- Diff: modules/core/src/main/java/org/apache/fluo/core/impl/TransactionImpl.java --- @@ -596,9 +596,17 @@ public CommitData createCommitData

[GitHub] incubator-fluo issue #832: closes #809 - cleanup buffered data

2017-05-09 Thread mjwall
Github user mjwall commented on the issue: https://github.com/apache/incubator-fluo/pull/832 same errors with mvn verify locally, so I'll spend some time trying to understand what is failing --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-fluo issue #832: closes #809 - cleanup buffered data

2017-05-09 Thread mjwall
Github user mjwall commented on the issue: https://github.com/apache/incubator-fluo/pull/832 mvn test ran fine locally, trying mvn verify to see if I get the same errors as travis --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-fluo pull request #832: closes #809 - cleanup buffered data

2017-05-09 Thread mjwall
Github user mjwall commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/832#discussion_r115542684 --- Diff: modules/core/src/main/java/org/apache/fluo/core/impl/TransactionImpl.java --- @@ -596,9 +597,14 @@ public CommitData createCommitData