[GitHub] mikewalch commented on issue #290: Investigate Mesos as an option for running Fluo

2017-11-06 Thread GitBox
mikewalch commented on issue #290: Investigate Mesos as an option for running Fluo URL: https://github.com/apache/fluo/issues/290#issuecomment-342205376 Closed by #883 This is an automated message from the Apache Git Service.

[GitHub] ctubbsii commented on a change in pull request #966: fluo-env.sh now supports FLUO_JAVA_OPTS set in environment

2017-11-06 Thread GitBox
ctubbsii commented on a change in pull request #966: fluo-env.sh now supports FLUO_JAVA_OPTS set in environment URL: https://github.com/apache/fluo/pull/966#discussion_r149233571 ## File path: modules/distribution/src/main/config/fluo-env.sh ## @@ -35,7 +35,12 @@ export F

[GitHub] jkosh44 commented on issue #759: Add scanner over multiple Spans

2017-11-07 Thread GitBox
jkosh44 commented on issue #759: Add scanner over multiple Spans URL: https://github.com/apache/fluo/issues/759#issuecomment-342496001 Is this still relevant? If so I'd like to give it a shot This is an automated message from

[GitHub] srikailash commented on a change in pull request #966: fluo-env.sh now supports FLUO_JAVA_OPTS set in environment

2017-11-07 Thread GitBox
srikailash commented on a change in pull request #966: fluo-env.sh now supports FLUO_JAVA_OPTS set in environment URL: https://github.com/apache/fluo/pull/966#discussion_r149472157 ## File path: modules/distribution/src/main/config/fluo-env.sh ## @@ -35,7 +35,12 @@ export

[GitHub] ctubbsii commented on a change in pull request #966: fluo-env.sh now supports FLUO_JAVA_OPTS set in environment

2017-11-07 Thread GitBox
ctubbsii commented on a change in pull request #966: fluo-env.sh now supports FLUO_JAVA_OPTS set in environment URL: https://github.com/apache/fluo/pull/966#discussion_r149523554 ## File path: modules/distribution/src/main/config/fluo-env.sh ## @@ -35,7 +35,11 @@ export F

[GitHub] keith-turner commented on issue #759: Add scanner over multiple Spans

2017-11-07 Thread GitBox
keith-turner commented on issue #759: Add scanner over multiple Spans URL: https://github.com/apache/fluo/issues/759#issuecomment-34260 @jkosh44 I am uncertain about this issue at the moment. But I am very uncertain about the API. There was the work done in #923 which should be consid

[GitHub] keith-turner commented on issue #759: Add scanner over multiple Spans

2017-11-07 Thread GitBox
keith-turner commented on issue #759: Add scanner over multiple Spans URL: https://github.com/apache/fluo/issues/759#issuecomment-34260 @jkosh44 I am uncertain about this issue at the moment. There was the work done in #923 which should be considered. Also the scanner builder API has

[GitHub] keith-turner closed issue #954: Support passing java options via enviroment variable

2017-11-08 Thread GitBox
keith-turner closed issue #954: Support passing java options via enviroment variable URL: https://github.com/apache/fluo/issues/954 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] keith-turner commented on issue #966: fluo-env.sh now supports FLUO_JAVA_OPTS set in environment

2017-11-08 Thread GitBox
keith-turner commented on issue #966: fluo-env.sh now supports FLUO_JAVA_OPTS set in environment URL: https://github.com/apache/fluo/pull/966#issuecomment-342848860 @srikailash thanks for the contribution. It seems the email address you used in the commit is not linked to your github acco

[GitHub] keith-turner commented on issue #106: Update basic-read-write.md

2017-11-08 Thread GitBox
keith-turner commented on issue #106: Update basic-read-write.md URL: https://github.com/apache/fluo-website/pull/106#issuecomment-342850536 @milleruntime thanks for the contribution. If you would like to be added to Fluo's [people](https://fluo.apache.org/people) page you can give m

[GitHub] llvieira commented on a change in pull request #108: Fix TODO for tour exercise

2017-11-08 Thread GitBox
llvieira commented on a change in pull request #108: Fix TODO for tour exercise URL: https://github.com/apache/fluo-website/pull/108#discussion_r149713422 ## File path: tour/exercise-1.md ## @@ -198,8 +198,7 @@ u:http://oldnews.com/a23 uri hash a6c4d1f Now that you have

[GitHub] milleruntime commented on issue #106: Update basic-read-write.md

2017-11-08 Thread GitBox
milleruntime commented on issue #106: Update basic-read-write.md URL: https://github.com/apache/fluo-website/pull/106#issuecomment-342873692 Although I wish it was a more useful contribution, I can add myself. Yes I am OK with a twitter mention. Twitter handle: millertimefatty --

[GitHub] milleruntime opened a new pull request #109: Add Michael Miller to contributors

2017-11-08 Thread GitBox
milleruntime opened a new pull request #109: Add Michael Miller to contributors URL: https://github.com/apache/fluo-website/pull/109 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] mikewalch closed pull request #109: Add Michael Miller to contributors

2017-11-08 Thread GitBox
mikewalch closed pull request #109: Add Michael Miller to contributors URL: https://github.com/apache/fluo-website/pull/109 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] mikewalch closed pull request #966: fluo-env.sh now supports FLUO_JAVA_OPTS set in environment

2017-11-08 Thread GitBox
mikewalch closed pull request #966: fluo-env.sh now supports FLUO_JAVA_OPTS set in environment URL: https://github.com/apache/fluo/pull/966 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] keith-turner opened a new issue #967: Prototype adding async get methods to Transaction

2017-11-08 Thread GitBox
keith-turner opened a new issue #967: Prototype adding async get methods to Transaction URL: https://github.com/apache/fluo/issues/967 [Fetching multiple cells](http://fluo.apache.org/tour/multi-get/) in the Fluo tour describes `get` methods that Fluo has to quickly read multiple cells.

[GitHub] keith-turner opened a new issue #968: Develop design for asynchronous observers

2017-11-08 Thread GitBox
keith-turner opened a new issue #968: Develop design for asynchronous observers URL: https://github.com/apache/fluo/issues/968 Currently when a Fluo worker thread executes an observer, the execution flow looks something like the following. * Worker Thread 1 : execute Observer proces

[GitHub] srikailash commented on issue #966: fluo-env.sh now supports FLUO_JAVA_OPTS set in environment

2017-11-08 Thread GitBox
srikailash commented on issue #966: fluo-env.sh now supports FLUO_JAVA_OPTS set in environment URL: https://github.com/apache/fluo/pull/966#issuecomment-342956986 Thanks for the community for helping me setup fluo . Looking forward for more pull requests . Regarding twitter usage : I d

[GitHub] srikailash opened a new pull request #110: added srikailash to contributors list

2017-11-08 Thread GitBox
srikailash opened a new pull request #110: added srikailash to contributors list URL: https://github.com/apache/fluo-website/pull/110 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] keith-turner commented on issue #110: added srikailash to contributors list

2017-11-08 Thread GitBox
keith-turner commented on issue #110: added srikailash to contributors list URL: https://github.com/apache/fluo-website/pull/110#issuecomment-342983436 Merged in bf00c5c67871ac0633ae122c452b10407095a880 This is an automated me

[GitHub] keith-turner closed pull request #110: added srikailash to contributors list

2017-11-08 Thread GitBox
keith-turner closed pull request #110: added srikailash to contributors list URL: https://github.com/apache/fluo-website/pull/110 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreig

[GitHub] keith-turner commented on issue #108: Fix TODO for tour exercise

2017-11-08 Thread GitBox
keith-turner commented on issue #108: Fix TODO for tour exercise URL: https://github.com/apache/fluo-website/pull/108#issuecomment-342983584 Merged in d826227a06928678e1615e9eae5a3fe847120065 This is an automated message from

[GitHub] keith-turner closed pull request #108: Fix TODO for tour exercise

2017-11-08 Thread GitBox
keith-turner closed pull request #108: Fix TODO for tour exercise URL: https://github.com/apache/fluo-website/pull/108 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull requ

[GitHub] keith-turner commented on issue #759: Add scanner over multiple Spans

2017-11-08 Thread GitBox
keith-turner commented on issue #759: Add scanner over multiple Spans URL: https://github.com/apache/fluo/issues/759#issuecomment-342988742 @jkosh44 would you be interested in prototyping #967 ? Its very open ended, I am not even sure what the best way to implement it is. If you are inter

[GitHub] jkosh44 commented on issue #759: Add scanner over multiple Spans

2017-11-09 Thread GitBox
jkosh44 commented on issue #759: Add scanner over multiple Spans URL: https://github.com/apache/fluo/issues/759#issuecomment-343327933 I'd definitely be interested in giving it a shot. This is an automated message from the Apa

[GitHub] jkosh44 commented on issue #967: Prototype adding async get methods to Transaction

2017-11-09 Thread GitBox
jkosh44 commented on issue #967: Prototype adding async get methods to Transaction URL: https://github.com/apache/fluo/issues/967#issuecomment-343328187 I'd like to try and put together some prototypes. This is an automated m

[GitHub] ctubbsii commented on issue #960: Use NewTableConfiguration (and also trivial code cleanup)

2017-11-09 Thread GitBox
ctubbsii commented on issue #960: Use NewTableConfiguration (and also trivial code cleanup) URL: https://github.com/apache/fluo/pull/960#issuecomment-343329664 Added commit to cover feedback from review. Will rebase and squash some of these together, upon review approval, just to clean up

[GitHub] keith-turner commented on issue #967: Prototype adding async get methods to Transaction

2017-11-09 Thread GitBox
keith-turner commented on issue #967: Prototype adding async get methods to Transaction URL: https://github.com/apache/fluo/issues/967#issuecomment-343345665 I have thought about two possible approaches to prototyping this. Both approaches would have the following. * An internal

[GitHub] keith-turner commented on a change in pull request #960: Use NewTableConfiguration (and also trivial code cleanup)

2017-11-09 Thread GitBox
keith-turner commented on a change in pull request #960: Use NewTableConfiguration (and also trivial code cleanup) URL: https://github.com/apache/fluo/pull/960#discussion_r150135795 ## File path: modules/accumulo/src/main/java/org/apache/fluo/accumulo/util/AccumuloProps.java #

[GitHub] keith-turner commented on a change in pull request #960: Use NewTableConfiguration (and also trivial code cleanup)

2017-11-09 Thread GitBox
keith-turner commented on a change in pull request #960: Use NewTableConfiguration (and also trivial code cleanup) URL: https://github.com/apache/fluo/pull/960#discussion_r150136202 ## File path: modules/integration/src/test/java/org/apache/fluo/integration/client/FluoAdminImplIT.j

[GitHub] keith-turner commented on a change in pull request #960: Use NewTableConfiguration (and also trivial code cleanup)

2017-11-09 Thread GitBox
keith-turner commented on a change in pull request #960: Use NewTableConfiguration (and also trivial code cleanup) URL: https://github.com/apache/fluo/pull/960#discussion_r150136309 ## File path: modules/integration/src/test/java/org/apache/fluo/integration/client/FluoAdminImplIT.j

[GitHub] ctubbsii commented on a change in pull request #960: Use NewTableConfiguration (and also trivial code cleanup)

2017-11-09 Thread GitBox
ctubbsii commented on a change in pull request #960: Use NewTableConfiguration (and also trivial code cleanup) URL: https://github.com/apache/fluo/pull/960#discussion_r150147137 ## File path: modules/integration/src/test/java/org/apache/fluo/integration/client/FluoAdminImplIT.java

[GitHub] keith-turner commented on a change in pull request #960: Use NewTableConfiguration (and also trivial code cleanup)

2017-11-09 Thread GitBox
keith-turner commented on a change in pull request #960: Use NewTableConfiguration (and also trivial code cleanup) URL: https://github.com/apache/fluo/pull/960#discussion_r150155003 ## File path: modules/integration/src/test/java/org/apache/fluo/integration/client/FluoAdminImplIT.j

[GitHub] asfgit closed pull request #960: Use NewTableConfiguration (and also trivial code cleanup)

2017-11-10 Thread GitBox
asfgit closed pull request #960: Use NewTableConfiguration (and also trivial code cleanup) URL: https://github.com/apache/fluo/pull/960 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] srikailash commented on issue #530: Improve 'fluo list' command to show more information

2017-11-11 Thread GitBox
srikailash commented on issue #530: Improve 'fluo list' command to show more information URL: https://github.com/apache/fluo/issues/530#issuecomment-343697432 Hey mikewalch , i am interested to work on this issue . This is a

[GitHub] jkosh44 commented on issue #967: Prototype adding async get methods to Transaction

2017-11-12 Thread GitBox
jkosh44 commented on issue #967: Prototype adding async get methods to Transaction URL: https://github.com/apache/fluo/issues/967#issuecomment-343761567 Hi Keith, Sorry I haven't been able to look at this until today. My first question is, should I be implementing these methods in t

[GitHub] keith-turner commented on issue #967: Prototype adding async get methods to Transaction

2017-11-13 Thread GitBox
keith-turner commented on issue #967: Prototype adding async get methods to Transaction URL: https://github.com/apache/fluo/issues/967#issuecomment-343997082 You could add a default method to SnapShotBase like the following. ```java default CompletableFuture getsAsync(String ro

[GitHub] jkosh44 commented on issue #967: Prototype adding async get methods to Transaction

2017-11-13 Thread GitBox
jkosh44 commented on issue #967: Prototype adding async get methods to Transaction URL: https://github.com/apache/fluo/issues/967#issuecomment-344105066 I'm a bit new to multi-threading so apologies if any of these questions seem basic. I think I understand your idea and am going to

[GitHub] jkosh44 opened a new pull request #969: [WIP] Issue-967

2017-11-13 Thread GitBox
jkosh44 opened a new pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969 Initial implementation for getsAsync() Created the following classes: AsyncReader TransactionImplTest This is an

[GitHub] jkosh44 commented on issue #969: [WIP] Issue-967

2017-11-13 Thread GitBox
jkosh44 commented on issue #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#issuecomment-344150332 This is my first pass at implementing getsAsync, though I haven't had a good chance to look it over and test properly. I started to try and test it but was having trouble cr

[GitHub] jkosh44 commented on issue #969: [WIP] Issue-967

2017-11-14 Thread GitBox
jkosh44 commented on issue #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#issuecomment-344259629 Also need to implement close() still This is an automated message from the Apache Git Service. To respond to

[GitHub] keith-turner commented on issue #967: Prototype adding async get methods to Transaction

2017-11-14 Thread GitBox
keith-turner commented on issue #967: Prototype adding async get methods to Transaction URL: https://github.com/apache/fluo/issues/967#issuecomment-344298140 @jkosh44 there are two problems with the `supplyAsync(() -> gets(row, column))` approach. First, the task will execute in a thread

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-14 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r150888175 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/AsyncReader.java ## @@ -0,0 +1,81 @@ +/* + * Licensed to the Apach

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-14 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r150899362 ## File path: modules/core/src/test/java/org/apache/fluo/core/impl/TransactionImplTest.java ## @@ -0,0 +1,55 @@ +/* + * Licensed to t

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-14 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r150886054 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/AsyncReader.java ## @@ -0,0 +1,81 @@ +/* + * Licensed to the Apach

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-14 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r150895303 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/AsyncReader.java ## @@ -0,0 +1,81 @@ +/* + * Licensed to the Apach

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-14 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r150892232 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/AsyncReader.java ## @@ -0,0 +1,81 @@ +/* + * Licensed to the Apach

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-14 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r150892232 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/AsyncReader.java ## @@ -0,0 +1,81 @@ +/* + * Licensed to the Apach

[GitHub] keith-turner commented on issue #967: Prototype adding async get methods to Transaction

2017-11-14 Thread GitBox
keith-turner commented on issue #967: Prototype adding async get methods to Transaction URL: https://github.com/apache/fluo/issues/967#issuecomment-344298140 @jkosh44 there are two problems with the `supplyAsync(() -> gets(row, column))` approach. First, the task will execute in a thread

[GitHub] keith-turner commented on issue #530: Improve 'fluo list' command to show more information

2017-11-14 Thread GitBox
keith-turner commented on issue #530: Improve 'fluo list' command to show more information URL: https://github.com/apache/fluo/issues/530#issuecomment-344331957 @mikewalch will you be able to provide @srikailash with assistance on this issue? If not, I may be able to. ---

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-16 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r151454155 ## File path: modules/api/src/main/java/org/apache/fluo/api/client/SnapshotBase.java ## @@ -165,4 +167,14 @@ * @return transactio

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-16 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r151452716 ## File path: modules/integration/src/test/java/org/apache/fluo/integration/impl/TransactionImplIT.java ## @@ -0,0 +1,48 @@ +/* + * L

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-16 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r151450761 ## File path: modules/integration/src/test/java/org/apache/fluo/integration/impl/TransactionImplIT.java ## @@ -0,0 +1,48 @@ +/* + * L

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-16 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r151454496 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/AsyncReader.java ## @@ -0,0 +1,86 @@ +/* + * Licensed to the Apach

[GitHub] asfgit closed pull request #955: Update plugin versions in parent POM

2017-11-16 Thread GitBox
asfgit closed pull request #955: Update plugin versions in parent POM URL: https://github.com/apache/fluo/pull/955 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] ctubbsii opened a new pull request #970: Revert back to formatter-maven-plugin 2.7.0

2017-11-17 Thread GitBox
ctubbsii opened a new pull request #970: Revert back to formatter-maven-plugin 2.7.0 URL: https://github.com/apache/fluo/pull/970 Version 2.7.1 had an issue in revelc/formatter-maven-plugin#260 This reverts back to version 2.7.0, which seems to work fine with Fluo --

[GitHub] mikewalch commented on issue #530: Improve 'fluo list' command to show more information

2017-11-17 Thread GitBox
mikewalch commented on issue #530: Improve 'fluo list' command to show more information URL: https://github.com/apache/fluo/issues/530#issuecomment-345383475 @srikailash, I updated the description with some more information that will hopefully get you started.

[GitHub] asfgit closed pull request #970: Revert back to formatter-maven-plugin 2.7.0

2017-11-17 Thread GitBox
asfgit closed pull request #970: Revert back to formatter-maven-plugin 2.7.0 URL: https://github.com/apache/fluo/pull/970 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull r

[GitHub] jkosh44 commented on issue #969: [WIP] Issue-967

2017-11-18 Thread GitBox
jkosh44 commented on issue #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#issuecomment-345472055 Just a note, I noticed all of the get methods with Byte were called "get(...)" while all the get methods with Strings were called "gets(...)". I kept this consistent with th

[GitHub] kpm1985 commented on issue #970: Revert back to formatter-maven-plugin 2.7.0

2017-11-19 Thread GitBox
kpm1985 commented on issue #970: Revert back to formatter-maven-plugin 2.7.0 URL: https://github.com/apache/fluo/pull/970#issuecomment-345502391 I haven't had a chance to review but I support all changes, u sir are trustworthy On Nov 17, 2017 2:30 PM, "asfgit" wrote: > Merg

[GitHub] cjmctague opened a new issue #971: IT's error out when calling mvn verify

2017-11-19 Thread GitBox
cjmctague opened a new issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971 I'm trying to vote in the parent-pom release and I can't get `mvn verify` to run properly even before checking it against the new parent-pom It appears to be a bu

[GitHub] ctubbsii commented on issue #971: IT's error out when calling mvn verify

2017-11-19 Thread GitBox
ctubbsii commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345571121 Did you try doing a `mvn clean` first? I always run `mvn clean verify` in case any stuff in the target directories isn't properly reset for the next

[GitHub] ctubbsii commented on issue #971: IT's error out when calling mvn verify

2017-11-19 Thread GitBox
ctubbsii commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345571634 Also, the test in error seems to be FluoAdminImplIT. There were a few recent changes with that to fix some things. Make sure your workspace is up to

[GitHub] cjmctague commented on issue #971: IT's error out when calling mvn verify

2017-11-19 Thread GitBox
cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345572723 @ctubbsii Its getting the error in all the IT's. I just didn't copy them all as it's a long list. Those are the first ones and all the rest dis

[GitHub] ctubbsii commented on issue #971: IT's error out when calling mvn verify

2017-11-19 Thread GitBox
ctubbsii commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345574159 Ah, right, I should have paid more attention to the initial post. Have you tried checking to see if there's an old ZK running that won't die (`ps au

[GitHub] cjmctague commented on issue #971: IT's error out when calling mvn verify

2017-11-19 Thread GitBox
cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345575624 `ps aux | grep jav[a]` returns nothing ``` jps -ml 1023 sun.tools.jps.Jps -ml ``` Rebooted and still getting the same list of errors.

[GitHub] ctubbsii commented on issue #971: IT's error out when calling mvn verify

2017-11-19 Thread GitBox
ctubbsii commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345577154 Hmm. This is a tricky one and I'm running low on ideas. Just to throw some more ideas out there to try: You could try clearing out all of you

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-20 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r152021426 ## File path: modules/api/src/main/java/org/apache/fluo/api/client/SnapshotBase.java ## @@ -165,4 +167,22 @@ * @return transactio

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-20 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r152021014 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/TransactionImpl.java ## @@ -265,6 +269,42 @@ public Bytes get(Byte

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-20 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r152018995 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/AsyncReader.java ## @@ -0,0 +1,100 @@ +/* + * Licensed to the Apac

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-20 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r152020031 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/AsyncReader.java ## @@ -0,0 +1,100 @@ +/* + * Licensed to the Apac

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-20 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r152019453 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/AsyncReader.java ## @@ -0,0 +1,100 @@ +/* + * Licensed to the Apac

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-20 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r152024441 ## File path: modules/integration/src/test/java/org/apache/fluo/integration/impl/TransactionImplIT.java ## @@ -0,0 +1,48 @@ +/* + * L

[GitHub] keith-turner commented on issue #969: [WIP] Issue-967

2017-11-20 Thread GitBox
keith-turner commented on issue #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#issuecomment-345737294 @jkosh44 I noticed your branch is based off an older commit. I usually add a git remote called upstream like the following. ```bash git remote add upstream ht

[GitHub] keith-turner commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
keith-turner commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345743469 Are there any stack trace lines after the messages like `Caused by: org.apache.zookeeper.KeeperException$NoAuthException: KeeperErrorCode = NoAu

[GitHub] cjmctague commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345753824 @ctubbsii No luck with any of that sadly. Be right back gonna toss my laptop off the roof. @keith-turner Is this what you were looki

[GitHub] keith-turner commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
keith-turner commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345755754 Try `grep -A 10 "NoAuth for" /tmp/fluo-build.log | head -20` This is an automa

[GitHub] keith-turner commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
keith-turner commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345756894 @cjmctague yeah you can email me the log. If you can bzip2 or gzip it. bzip will probably compress better. -

[GitHub] cjmctague commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345759461 @keith-turner ``` grep -A 10 "NoAuth for" /tmp/fluo-build.log | head -20 java.lang.RuntimeException: org.apache.zookeeper.KeeperException

[GitHub] keith-turner commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
keith-turner commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345773493 Ugh, so -`X` did not show the stack traces. Maybe the `-e` option will. Could try `mvn -X -e -l /tmp/fluo-build.log` Also there may be

[GitHub] keith-turner commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
keith-turner commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345819064 Something else to look at is `modules/integration/target/accumulo-maven-plugin/it-instance-maven/logs/Zoo*`. These files are for the Zookeeper

[GitHub] cjmctague commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345753824 @ctubbsii No luck with any of that sadly. Be right back gonna toss my laptop off the roof. @keith-turner Is this what you were looki

[GitHub] ctubbsii commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
ctubbsii commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345874822 Another thing you could try (very much doubt this is the issue) is disabling your firewall and/or selinux (if that applies to you). The problem cou

[GitHub] jkosh44 commented on a change in pull request #969: [WIP] Issue-967

2017-11-20 Thread GitBox
jkosh44 commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r152152031 ## File path: modules/api/src/main/java/org/apache/fluo/api/client/SnapshotBase.java ## @@ -165,4 +167,22 @@ * @return transactions st

[GitHub] jkosh44 commented on a change in pull request #969: [WIP] Issue-967

2017-11-20 Thread GitBox
jkosh44 commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r152152267 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/AsyncReader.java ## @@ -0,0 +1,100 @@ +/* + * Licensed to the Apache So

[GitHub] jkosh44 commented on a change in pull request #969: [WIP] Issue-967

2017-11-20 Thread GitBox
jkosh44 commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r152154650 ## File path: modules/integration/src/test/java/org/apache/fluo/integration/impl/TransactionImplIT.java ## @@ -0,0 +1,48 @@ +/* + * Licens

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-20 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r152158725 ## File path: modules/api/src/main/java/org/apache/fluo/api/client/SnapshotBase.java ## @@ -165,4 +167,22 @@ * @return transactio

[GitHub] jkosh44 commented on issue #969: [WIP] Issue-967

2017-11-20 Thread GitBox
jkosh44 commented on issue #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#issuecomment-345888273 Is it too late to follow the git upstream steps for this issue? This is an automated message from the Apache

[GitHub] cjmctague commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345888543 @keith-turner The `-e` flag didn't change anything. Still not finding any trace in the log file. In the `failsafe-reports` it has all the s

[GitHub] cjmctague commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345888617 @ctubbsii On a Mac and I disabled to firewall and nothing changed. This is an

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-20 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r152159759 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/AsyncReader.java ## @@ -0,0 +1,100 @@ +/* + * Licensed to the Apac

[GitHub] keith-turner commented on issue #969: [WIP] Issue-967

2017-11-20 Thread GitBox
keith-turner commented on issue #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#issuecomment-345889897 > Is it too late to follow the git upstream steps for this issue? No, you can try. If you are new to git I would recommend experimenting in another branch. Try

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-20 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r152160862 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/AsyncReader.java ## @@ -0,0 +1,101 @@ +/* + * Licensed to the Apac

[GitHub] keith-turner commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
keith-turner commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345891355 @cjmctague what about the zookeeper server logs, anything there? This is an aut

[GitHub] cjmctague commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345893991 @keith-turner Sorry I forgot about that. There is a lot more to it but this is the gist of it from what I can tell ``` 2017-11-20 20:42:0

[GitHub] cjmctague commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345893991 @keith-turner Sorry I forgot about that. There is a lot more to it but this is the gist of it from what I can tell ``` 2017-11-20 20:42:0

[GitHub] cjmctague commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345893991 @keith-turner Sorry I forgot about that. There is a lot more to it but this is the gist of it from what I can tell ``` 2017-11-20 20:42:0

[GitHub] cjmctague commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345893991 @keith-turner Sorry I forgot about that. There is a lot more to it but this is the gist of it from what I can tell ``` 2017-11-20 20:42:0

[GitHub] keith-turner commented on a change in pull request #969: [WIP] Issue-967

2017-11-20 Thread GitBox
keith-turner commented on a change in pull request #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#discussion_r152164105 ## File path: modules/integration/src/test/java/org/apache/fluo/integration/impl/TransactionImplIT.java ## @@ -0,0 +1,48 @@ +/* + * L

[GitHub] keith-turner commented on issue #971: IT's error out when calling mvn verify

2017-11-20 Thread GitBox
keith-turner commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345899607 @cjmctague can you find the test org.apache.fluo.integration.impl.ZKSecretIT and delete it? I am wondering if that test is mucking things up

<    1   2   3   4   5   6   7   8   9   10   >