[GitHub] kpm1985 commented on issue #941: WIP: Fix for FLUO-938

kpm1985 commented on issue #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941#issuecomment-337762153 @keith-turner thank you, I made a few changes to the constructor On Oct 18, 2017 1:58 PM, "Keith Turner" wrote: > @kpm1985

[GitHub] asfgit closed issue #938: Make transaction info cache size configurable.

asfgit closed issue #938: Make transaction info cache size configurable. URL: https://github.com/apache/fluo/issues/938 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] keith-turner closed pull request #941: WIP: Fix for FLUO-938

keith-turner closed pull request #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941 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 (from a

[GitHub] kpm1985 commented on issue #939: Make TransactorCache size configurable

kpm1985 commented on issue #939: Make TransactorCache size configurable URL: https://github.com/apache/fluo/issues/939#issuecomment-337768966 Would you like me to take this or work on something else @keith-turner ? I don't want to be a hog, and there was a person on the mailing list

[GitHub] keith-turner commented on issue #939: Make TransactorCache size configurable

keith-turner commented on issue #939: Make TransactorCache size configurable URL: https://github.com/apache/fluo/issues/939#issuecomment-337768884 The commit 650f044b7db16e732cfbf32f484f05ec34837df8 for #938 can be used as a template for this issue.

[GitHub] kpm1985 opened a new pull request #945: Remove redundant call whos value is not used anyways

kpm1985 opened a new pull request #945: Remove redundant call whos value is not used anyways URL: https://github.com/apache/fluo/pull/945 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] kpm1985 commented on issue #942: VisibilityCache constructor is out of date

kpm1985 commented on issue #942: VisibilityCache constructor is out of date URL: https://github.com/apache/fluo/issues/942#issuecomment-337826720 While I notice I did not earse TxInfoCache.CACHE_TIMEOUT_MIN, it is still cleaner to use the values from FluoConfigurationImpl and the following

[GitHub] kpm1985 opened a new issue #942: VisibilityCache constructor is out of date

kpm1985 opened a new issue #942: VisibilityCache constructor is out of date URL: https://github.com/apache/fluo/issues/942 ``` private final Cache visCache; VisibilityCache() { visCache =

[GitHub] keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938

keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941#discussion_r145514539 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/FluoConfigurationImpl.java ## @@ -102,6 +104,51 @@ public

[GitHub] keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938

keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941#discussion_r145514924 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/FluoConfigurationImpl.java ## @@ -102,6 +104,51 @@ public

[GitHub] keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938

keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941#discussion_r145515154 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/FluoConfigurationImpl.java ## @@ -102,6 +104,51 @@ public

[GitHub] keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938

keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941#discussion_r145514670 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/FluoConfigurationImpl.java ## @@ -102,6 +104,51 @@ public

[GitHub] keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938

keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941#discussion_r145514042 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/FluoConfigurationImpl.java ## @@ -102,6 +104,51 @@ public

[GitHub] kpm1985 commented on issue #941: WIP: Fix for FLUO-938

kpm1985 commented on issue #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941#issuecomment-337706073 I changed the data type for the default property to simply be a long instead of an int. Same functionality, no more boxing or casting and can use conf.getLong().

[GitHub] keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938

keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941#discussion_r145524430 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/FluoConfigurationImpl.java ## @@ -102,6 +104,46 @@ public

[GitHub] keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938

keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941#discussion_r145524525 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/FluoConfigurationImpl.java ## @@ -102,6 +104,46 @@ public

[GitHub] keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938

keith-turner commented on a change in pull request #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941#discussion_r145524292 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/FluoConfigurationImpl.java ## @@ -102,6 +104,46 @@ public

[GitHub] ctubbsii opened a new pull request #955: Update plugin versions in parent POM

ctubbsii opened a new pull request #955: Update plugin versions in parent POM URL: https://github.com/apache/fluo/pull/955 Update plugin versions, especially the formatter (to fix bug in javadoc formatting). This is an

[GitHub] kpm1985 commented on issue #951: FLUO-950 Make VisibilityCache customizable

kpm1985 commented on issue #951: FLUO-950 Make VisibilityCache customizable URL: https://github.com/apache/fluo/pull/951#issuecomment-338787690 @keith-turner Nice catch! This is an automated message from the Apache Git

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

keith-turner opened a new issue #954: Support passing java options via enviroment variable URL: https://github.com/apache/fluo/issues/954 In order to support apache/fluo-docker#2 it would be nice if the fluo scripts supported passing java options via environment variables.

[GitHub] jkosh44 commented on issue #946: Create a column buffer class for GarbageCollectionIterator

jkosh44 commented on issue #946: Create a column buffer class for GarbageCollectionIterator URL: https://github.com/apache/fluo/issues/946#issuecomment-338527932 Hey @keith-turner I pushed some code and added comments about things I wasn't sure about. Can you take a look at it and let me

[GitHub] kpm1985 commented on issue #951: FLUO-950 Make VisibilityCache customizable

kpm1985 commented on issue #951: FLUO-950 Make VisibilityCache customizable URL: https://github.com/apache/fluo/pull/951#issuecomment-338538784 @keith-turner I've pushed some changes. Build fails but I looked it seems unrelated (timeout stuff again). On Oct 22, 2017 6:58 PM,

[GitHub] ctubbsii opened a new pull request #5: [WIP] DO NOT MERGE - Initial implementation

ctubbsii opened a new pull request #5: [WIP] DO NOT MERGE - Initial implementation URL: https://github.com/apache/fluo-bytes/pull/5 This is a work in progress. It is not quite ready to merge. I want to make it available for review/feedback before I make any more progress, to make sure

[GitHub] kpm1985 commented on issue #941: WIP: Fix for FLUO-938

kpm1985 commented on issue #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941#issuecomment-337745507 @keith-turner Thank you, I've modified the constructor to take the new methods This is an automated

[GitHub] kpm1985 commented on issue #941: WIP: Fix for FLUO-938

kpm1985 commented on issue #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941#issuecomment-337722536 @keith-turner I get what the intent was. Hopefully QED On Oct 18, 2017 12:59 PM, "Keith Turner" wrote: > *@keith-turner*

[GitHub] keith-turner commented on issue #941: WIP: Fix for FLUO-938

keith-turner commented on issue #941: WIP: Fix for FLUO-938 URL: https://github.com/apache/fluo/pull/941#issuecomment-337725888 @kpm1985 these changes look great. After make the cache code call these new config methods, this will be ready to merge.

[GitHub] kpm1985 commented on issue #943: FLUO-942 The constructor for VisibilityCache now uses the values from?

kpm1985 commented on issue #943: FLUO-942 The constructor for VisibilityCache now uses the values from? URL: https://github.com/apache/fluo/pull/943#issuecomment-337990736 @keith-turner I have fixed my git config --global user.name and user.email. Thank you very much for the heads up.

[GitHub] keith-turner commented on a change in pull request #5: [WIP] DO NOT MERGE - Initial implementation

keith-turner commented on a change in pull request #5: [WIP] DO NOT MERGE - Initial implementation URL: https://github.com/apache/fluo-bytes/pull/5#discussion_r145786386 ## File path: src/main/java/org/apache/bytes/Bytes.java ## @@ -0,0 +1,452 @@ +/* + * Licensed to the

[GitHub] ctubbsii commented on a change in pull request #5: [WIP] DO NOT MERGE - Initial implementation

ctubbsii commented on a change in pull request #5: [WIP] DO NOT MERGE - Initial implementation URL: https://github.com/apache/fluo-bytes/pull/5#discussion_r145805466 ## File path: src/main/java/org/apache/bytes/Bytes.java ## @@ -0,0 +1,452 @@ +/* + * Licensed to the

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

keith-turner commented on a change in pull request #953: fixes #917 Added read locks URL: https://github.com/apache/fluo/pull/953#discussion_r146361167 ## File path: modules/integration/src/test/resources/log4j.properties ## @@ -36,3 +36,5 @@

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

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

[GitHub] keith-turner commented on a change in pull request #951: FLUO-950 Make VisibilityCache customizable

keith-turner commented on a change in pull request #951: FLUO-950 Make VisibilityCache customizable URL: https://github.com/apache/fluo/pull/951#discussion_r146360582 ## File path: modules/core/src/main/java/org/apache/fluo/core/impl/VisibilityCache.java ## @@ -46,11

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

keith-turner commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-347248752 > After removing the ZKSecretIT the "NoAuth" errors have gone away. Is there a way to for this IT to go last or should we just fix the "Auth"

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

keith-turner commented on issue #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#issuecomment-347307505 @jkosh44 these changes are looking good. I am still puzzling out the async story for Fluo. Currently there are a few areas I have identified and opened issue about.

[GitHub] keith-turner commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture

keith-turner commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture URL: https://github.com/apache/fluo/issues/722#issuecomment-347562463 @jkosh44 this will be a very involved change. I can assist as needed, let me know if you have any questions. You

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

keith-turner commented on issue #969: [WIP] Issue-967 URL: https://github.com/apache/fluo/pull/969#issuecomment-347557885 > On an unrelated note, I feel like all my commits fail on the continuous-integration Travis CI build. Is there a specific reason? The builds on travis are not

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

cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-347401335 My environment currently is on a mid 2012 macOS laptop with the latest version that the issues are occurring on. I run`brew update` and `brew

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

cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-347635816 @keith-turner Removing the timeout rule just made the test never end. I just emailed you the jStack dump

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

keith-turner commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-347672768 I looked at the jstack you sent, it seems like the file contains 4 jstacks. Are those for 4 different test runs? It seems in two the of

[GitHub] srikailash commented on issue #530: Improve 'fluo list' command to print Fluo application start time

srikailash commented on issue #530: Improve 'fluo list' command to print Fluo application start time URL: https://github.com/apache/fluo/issues/530#issuecomment-346949735 @mikewalch working on this . Information seems fine , will update here if any thing is needed .

[GitHub] keith-turner opened a new issue #973: MiniFluo fails to start when running from mvn exec

keith-turner opened a new issue #973: MiniFluo fails to start when running from mvn exec URL: https://github.com/apache/fluo/issues/973 When trying to use maven exec to start a 1.2.0-SNAPSHOT instance of mini fluo it fails with the following exeception. I think what is happening is that

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

cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-348279132 @keith-turner Yea sorry those were two different test runs. This is an automated

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

cjmctague commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-348279132 @keith-turner Yea sorry those were four different test runs. This is an automated

[GitHub] keith-turner opened a new pull request #974: fixes #973 default fluo.dfs.root to empty string

keith-turner opened a new pull request #974: fixes #973 default fluo.dfs.root to empty string URL: https://github.com/apache/fluo/pull/974 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] keith-turner closed pull request #974: fixes #973 default fluo.dfs.root to empty string

keith-turner closed pull request #974: fixes #973 default fluo.dfs.root to empty string URL: https://github.com/apache/fluo/pull/974 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] jkosh44 commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture

jkosh44 commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture URL: https://github.com/apache/fluo/issues/722#issuecomment-347378967 Since I just got some experience using CompletableFutures I'd like to give this a shot.

[GitHub] ctubbsii closed pull request #972: Update parent to 3

ctubbsii closed pull request #972: Update parent to 3 URL: https://github.com/apache/fluo/pull/972 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 (from a fork),

[GitHub] ctubbsii opened a new pull request #145: Update parent to 3

ctubbsii opened a new pull request #145: Update parent to 3 URL: https://github.com/apache/fluo-recipes/pull/145 Two commits. The first updates the parent POM to version 3, and strips out tasks that are now in the parent POM. The second applies formatter changes based on the updated

[GitHub] ctubbsii opened a new pull request #972: Update parent to 3

ctubbsii opened a new pull request #972: Update parent to 3 URL: https://github.com/apache/fluo/pull/972 Two commits in this PR. The first updates the parent POM. The second applies formatting changes resulting from the new formatter in the new parent POM, and some manual whitespace

[GitHub] ctubbsii opened a new pull request #10: Update fluo-parent to version 3

ctubbsii opened a new pull request #10: Update fluo-parent to version 3 URL: https://github.com/apache/fluo-yarn/pull/10 * Update parent POM to fluo-parent version 3 * Apply formatter changes (usually would go in a separate commit, but the changeset was small enough that it didn't

[GitHub] jwonders commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture

jwonders commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture URL: https://github.com/apache/fluo/issues/722#issuecomment-348667973 Hi folks, I've been lurking around on the mailing lists for awhile as I haven't really had much time to start contributing.

[GitHub] jkosh44 commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture

jkosh44 commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture URL: https://github.com/apache/fluo/issues/722#issuecomment-349050151 Hey sorry I've had an extremely hectic week and haven't been able to look at this yet. I'm going to start on this tonight.

[GitHub] jkosh44 commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture

jkosh44 commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture URL: https://github.com/apache/fluo/issues/722#issuecomment-349184990 Did some analysis of where ListenableFuture is used and here's what I found: Classes that use ListenableFuture:

[GitHub] jwonders commented on issue #978: Use CompleteableFuture compose to centralize commit logic

jwonders commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-351738004 @jkosh44 I'm interested in looking at it. I probably can't get to it until this weekend rthough. Hopefully the

[GitHub] jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic

jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-351726809 If anyone's interested I started to try and see how this would look/work in code.

[GitHub] jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic

jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-351726809 If anyone's interested I started to try and see how this would look/work in code.

[GitHub] jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic

jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-351726809 If anyone's interested I started to try and see how this would look/work in code.

[GitHub] jkosh44 commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture

jkosh44 commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture URL: https://github.com/apache/fluo/issues/722#issuecomment-351151776 @jwonders I've been looking at your suggestions and had a question. I'm still unclear as to what AsynCommitResult's purpose

[GitHub] mikewalch opened a new pull request #12: Fixes #11 Apps are failing to run on cluster

mikewalch opened a new pull request #12: Fixes #11 Apps are failing to run on cluster URL: https://github.com/apache/fluo-yarn/pull/12 * Created new property so that 'fs.defaultFS' is set in hadoop config that is passed to YarnTwillRunnerService

[GitHub] mikewalch opened a new issue #11: Apps are failing to launch when running on cluster

mikewalch opened a new issue #11: Apps are failing to launch when running on cluster URL: https://github.com/apache/fluo-yarn/issues/11 I think this is due to Twill not storing the files it needs in HDFS because `fs.defaultFS` is not configured in Hadoop configuration that is passed to

[GitHub] keith-turner opened a new issue #9: Download from Apache mirrors instead of archive

keith-turner opened a new issue #9: Download from Apache mirrors instead of archive URL: https://github.com/apache/fluo-docker/issues/9 Currently the dockerfile downloads dependencies like Hadoop and Zookeeper from Apache's archive site. It would be much better to download from Apache

[GitHub] keith-turner commented on issue #9: Download from Apache mirrors instead of archive

keith-turner commented on issue #9: Download from Apache mirrors instead of archive URL: https://github.com/apache/fluo-docker/issues/9#issuecomment-352809208 Also the Accumulo docker file uses the technique mentioned in the comment

[GitHub] keith-turner commented on issue #978: Use CompleteableFuture compose to centralize commit logic

keith-turner commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-352818006 @jwonders the flow chart is really awesome! What did you use to create it? Would you be willing to contribute it? I

[GitHub] jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic

jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-351572833 @keith-turner I've been trying to think of a way to synthesize @jwonders idea of an AysncCommitResult and your idea of having

[GitHub] jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic

jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-351581495 I was going to try and see how this worked in actual code and I noticed that this would not cover the case where we commit and

[GitHub] jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic

jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-351564607 Just wanted to also copy some relevant comments from #722 by @jwonders, to avoid having to flip back and forth. >Assuming

[GitHub] jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic

jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-351573940 `AsyncCommitResult` would look like ``` public class AsyncCommitResult { private CommitData cd; private

[GitHub] ctubbsii commented on issue #978: Use CompleteableFuture compose to centralize commit logic

ctubbsii commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-351458579 @keith-turner that Gist isn't public. This is an automated

[GitHub] keith-turner commented on issue #978: Use CompleteableFuture compose to centralize commit logic

keith-turner commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-351458277 I created a [Gist](https://gist.github.com/keith-turner/387dc83332bc330fee673be14684899) containing some local

[GitHub] keith-turner commented on issue #978: Use CompleteableFuture compose to centralize commit logic

keith-turner commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-351460771 This is the commit code as it existed before being converted to an async process.

[GitHub] keith-turner commented on issue #978: Use CompleteableFuture compose to centralize commit logic

keith-turner commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-351458277 I created a [Gist](https://gist.github.com/keith-turner/387dc83332bc330fee673be146848994) containing some local

[GitHub] ctubbsii commented on issue #978: Use CompleteableFuture compose to centralize commit logic

ctubbsii commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-351458579 @keith-turner that Gist isn't public. This is an automated

[GitHub] jwonders commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture

jwonders commented on issue #722: Move from Guava ListenableFuture to Java 8 CompletableFuture URL: https://github.com/apache/fluo/issues/722#issuecomment-351266088 The purpose of `AsyncCommitResult` is to capture the result of the async operation. Terminal steps would be able to create

[GitHub] mikewalch closed issue #11: Apps are failing to launch when running on cluster

mikewalch closed issue #11: Apps are failing to launch when running on cluster URL: https://github.com/apache/fluo-yarn/issues/11 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] mikewalch closed pull request #12: Fixes #11 Apps are failing to run on cluster

mikewalch closed pull request #12: Fixes #11 Apps are failing to run on cluster URL: https://github.com/apache/fluo-yarn/pull/12 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] alanblueshift opened a new pull request #10: Fix to download from Apache mirrors instead of archive #9

alanblueshift opened a new pull request #10: Fix to download from Apache mirrors instead of archive #9 URL: https://github.com/apache/fluo-docker/pull/10 This is an automated message from the Apache Git Service. To respond

[GitHub] jwonders commented on issue #978: Use CompleteableFuture compose to centralize commit logic

jwonders commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-352944995 I used gliffy. Definitely willing to contribute it. Here is a gist of the json.

[GitHub] keith-turner closed pull request #10: Fix to download from Apache mirrors instead of archive #9

keith-turner closed pull request #10: Fix to download from Apache mirrors instead of archive #9 URL: https://github.com/apache/fluo-docker/pull/10 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:

[GitHub] keith-turner commented on issue #10: Fix to download from Apache mirrors instead of archive #9

keith-turner commented on issue #10: Fix to download from Apache mirrors instead of archive #9 URL: https://github.com/apache/fluo-docker/pull/10#issuecomment-353055580 @ctubbsii I opened #11 This is an automated message

[GitHub] jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic

jkosh44 commented on issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978#issuecomment-352326285 @keith-turner I haven't had a chance to look at the code in depth but it seems to achieve everything you were looking for.

[GitHub] keith-turner opened a new issue #978: Use CompleteableFuture compose to centralize commit logic

keith-turner opened a new issue #978: Use CompleteableFuture compose to centralize commit logic URL: https://github.com/apache/fluo/issues/978 Opening this issue for follow on work identified in the comments of #975 and #722.

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

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 @@ +/* + *

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

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

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

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 @@ +/* + *

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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:

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

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

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

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

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

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

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

keith-turner commented on issue #971: IT's error out when calling mvn verify URL: https://github.com/apache/fluo/issues/971#issuecomment-345900077 I noticed in the logs you emailed me that ZKSecretIT runs first and then everything after that fails. That test does modify auths in

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

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] jkosh44 commented on a change in pull request #969: [WIP] Issue-967

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

  1   2   3   4   5   6   7   8   9   10   >