[GitHub] kamaci opened a new pull request #988: Closed file system which is not eventually closed.

2017-12-22 Thread GitBox
kamaci opened a new pull request #988: Closed file system which is not eventually closed. URL: https://github.com/apache/fluo/pull/988 Closed file system. This is an automated message from the Apache Git Service. To respond

[GitHub] kpm1985 opened a new pull request #989: FLUO-768 Improve Oracle log message

2017-12-26 Thread GitBox
kpm1985 opened a new pull request #989: FLUO-768 Improve Oracle log message URL: https://github.com/apache/fluo/pull/989 This commit and change is an effort at making the log statement understandable to the user, based on user feedback.

[GitHub] kpm1985 commented on issue #989: FLUO-768 Improve Oracle log message

2017-12-26 Thread GitBox
kpm1985 commented on issue #989: FLUO-768 Improve Oracle log message URL: https://github.com/apache/fluo/pull/989#issuecomment-354002897 Thank you for the catch! Dyslexic mistake for sure. Just opened a corrected PR. On Dec 26, 2017 10:35 AM, "Mike Walch"

[GitHub] kpm1985 closed pull request #990: FLUO-786 Improve Oracle log message

2017-12-26 Thread GitBox
kpm1985 closed pull request #990: FLUO-786 Improve Oracle log message URL: https://github.com/apache/fluo/pull/990 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] kpm1985 opened a new pull request #990: FLUO-786 Improve Oracle log message

2017-12-26 Thread GitBox
kpm1985 opened a new pull request #990: FLUO-786 Improve Oracle log message URL: https://github.com/apache/fluo/pull/990 This addresses #786 based on user feedback. This is an automated message from the Apache Git Service. To

[GitHub] mikewalch opened a new pull request #121: Updated GitHub pages to latest version

2017-12-26 Thread GitBox
mikewalch opened a new pull request #121: Updated GitHub pages to latest version URL: https://github.com/apache/fluo-website/pull/121 * Removed variables in uses of redirect_to * Made a hard to see color in syntax highligting darker

[GitHub] keith-turner commented on issue #12: Checksum the download of the binaries

2017-12-21 Thread GitBox
keith-turner commented on issue #12: Checksum the download of the binaries URL: https://github.com/apache/fluo-docker/pull/12#issuecomment-353383679 The readme currently has a command for building with custom versions. It should be updated with the hash build args. Since its so long

[GitHub] ctubbsii commented on issue #11: Verify checksums for files downloaded from Apache mirrors

2017-12-20 Thread GitBox
ctubbsii commented on issue #11: Verify checksums for files downloaded from Apache mirrors URL: https://github.com/apache/fluo-docker/issues/11#issuecomment-353185731 Oh, also, the hardcoded solution saves us from accessing the apache.org site when not needed, which is the whole point of

[GitHub] keith-turner commented on issue #979: Make libthrift dependency "provided"

2017-12-20 Thread GitBox
keith-turner commented on issue #979: Make libthrift dependency "provided" URL: https://github.com/apache/fluo/pull/979#issuecomment-353195958 I would be in favor of experimenting with shading libthrift. Even if Fluo works with all currently released libthrift versions after 0.9.1 there

[GitHub] keith-turner commented on issue #11: Verify checksums for files downloaded from Apache mirrors

2017-12-20 Thread GitBox
keith-turner commented on issue #11: Verify checksums for files downloaded from Apache mirrors URL: https://github.com/apache/fluo-docker/issues/11#issuecomment-353187845 I think the hardcoded solution is ok. This is an

[GitHub] kpm1985 closed pull request #114: Add Alan Camillo as a contributor

2017-12-21 Thread GitBox
kpm1985 closed pull request #114: Add Alan Camillo as a contributor URL: https://github.com/apache/fluo-website/pull/114 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] kamaci opened a new pull request #996: Loggers are used instead to printing Throwable.

2018-01-02 Thread GitBox
kamaci opened a new pull request #996: Loggers are used instead to printing Throwable. URL: https://github.com/apache/fluo/pull/996 Loggers are used instead to printing Throwable. This is an automated message from the Apache

[GitHub] keith-turner commented on issue #992: Notifications never processed while running stress test

2018-01-03 Thread GitBox
keith-turner commented on issue #992: Notifications never processed while running stress test URL: https://github.com/apache/fluo/issues/992#issuecomment-355066495 I was able to confirm that NotificationTracker.requeue() not being synchronized while accessing a hashmap caused the problem.

[GitHub] keith-turner opened a new pull request #998: fixes #992 sync tracking of notifications

2018-01-03 Thread GitBox
keith-turner opened a new pull request #998: fixes #992 sync tracking of notifications URL: https://github.com/apache/fluo/pull/998 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] keith-turner closed issue #992: Notifications never processed while running stress test

2018-01-03 Thread GitBox
keith-turner closed issue #992: Notifications never processed while running stress test URL: https://github.com/apache/fluo/issues/992 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] keith-turner closed pull request #998: fixes #992 sync tracking of notifications

2018-01-03 Thread GitBox
keith-turner closed pull request #998: fixes #992 sync tracking of notifications URL: https://github.com/apache/fluo/pull/998 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

[GitHub] mikewalch commented on issue #129: #127 Provide instructions for verifying release

2018-02-26 Thread GitBox
mikewalch commented on issue #129: #127 Provide instructions for verifying release URL: https://github.com/apache/fluo-website/pull/129#issuecomment-368626353 @ctubbsii, I made a bunch of updates. Let me know what you think.

[GitHub] mikewalch commented on a change in pull request #129: #127 Provide instructions for verifying release

2018-02-26 Thread GitBox
mikewalch commented on a change in pull request #129: #127 Provide instructions for verifying release URL: https://github.com/apache/fluo-website/pull/129#discussion_r170748483 ## File path: pages/release-process.md ## @@ -146,68 +146,108 @@ When the vote passes on a

[GitHub] mikewalch commented on a change in pull request #129: #127 Provide instructions for verifying release

2018-02-26 Thread GitBox
mikewalch commented on a change in pull request #129: #127 Provide instructions for verifying release URL: https://github.com/apache/fluo-website/pull/129#discussion_r170752742 ## File path: pages/release-process.md ## @@ -146,68 +146,108 @@ When the vote passes on a

[GitHub] mikewalch commented on a change in pull request #129: #127 Provide instructions for verifying release

2018-02-26 Thread GitBox
mikewalch commented on a change in pull request #129: #127 Provide instructions for verifying release URL: https://github.com/apache/fluo-website/pull/129#discussion_r170748878 ## File path: pages/release-process.md ## @@ -146,68 +146,108 @@ When the vote passes on a

[GitHub] keith-turner commented on a change in pull request #129: #127 Provide instructions for verifying release

2018-02-26 Thread GitBox
keith-turner commented on a change in pull request #129: #127 Provide instructions for verifying release URL: https://github.com/apache/fluo-website/pull/129#discussion_r170751600 ## File path: pages/release-process.md ## @@ -144,6 +144,110 @@ When the vote passes on a

[GitHub] mikewalch commented on a change in pull request #129: #127 Provide instructions for verifying release

2018-02-26 Thread GitBox
mikewalch commented on a change in pull request #129: #127 Provide instructions for verifying release URL: https://github.com/apache/fluo-website/pull/129#discussion_r170760210 ## File path: pages/release-process.md ## @@ -146,68 +146,108 @@ When the vote passes on a

[GitHub] ctubbsii commented on a change in pull request #129: #127 Provide instructions for verifying release

2018-02-26 Thread GitBox
ctubbsii commented on a change in pull request #129: #127 Provide instructions for verifying release URL: https://github.com/apache/fluo-website/pull/129#discussion_r170720385 ## File path: pages/release-process.md ## @@ -146,68 +146,108 @@ When the vote passes on a

[GitHub] ctubbsii commented on a change in pull request #129: #127 Provide instructions for verifying release

2018-02-26 Thread GitBox
ctubbsii commented on a change in pull request #129: #127 Provide instructions for verifying release URL: https://github.com/apache/fluo-website/pull/129#discussion_r170719601 ## File path: pages/release-process.md ## @@ -146,68 +146,108 @@ When the vote passes on a

[GitHub] ctubbsii commented on a change in pull request #129: #127 Provide instructions for verifying release

2018-02-26 Thread GitBox
ctubbsii commented on a change in pull request #129: #127 Provide instructions for verifying release URL: https://github.com/apache/fluo-website/pull/129#discussion_r170719116 ## File path: pages/release-process.md ## @@ -146,68 +146,108 @@ When the vote passes on a

[GitHub] ctubbsii commented on a change in pull request #129: #127 Provide instructions for verifying release

2018-02-26 Thread GitBox
ctubbsii commented on a change in pull request #129: #127 Provide instructions for verifying release URL: https://github.com/apache/fluo-website/pull/129#discussion_r170719916 ## File path: pages/release-process.md ## @@ -146,68 +146,108 @@ When the vote passes on a

[GitHub] keith-turner commented on a change in pull request #1043: FLUO-1002 Create integration test for `FluoAdmin.remove()`

2018-06-20 Thread GitBox
keith-turner commented on a change in pull request #1043: FLUO-1002 Create integration test for `FluoAdmin.remove()` URL: https://github.com/apache/fluo/pull/1043#discussion_r196757745 ## File path:

[GitHub] keith-turner commented on a change in pull request #1043: FLUO-1002 Create integration test for `FluoAdmin.remove()`

2018-06-20 Thread GitBox
keith-turner commented on a change in pull request #1043: FLUO-1002 Create integration test for `FluoAdmin.remove()` URL: https://github.com/apache/fluo/pull/1043#discussion_r196760081 ## File path:

[GitHub] keith-turner commented on a change in pull request #1043: FLUO-1002 Create integration test for `FluoAdmin.remove()`

2018-06-20 Thread GitBox
keith-turner commented on a change in pull request #1043: FLUO-1002 Create integration test for `FluoAdmin.remove()` URL: https://github.com/apache/fluo/pull/1043#discussion_r196754986 ## File path: modules/core/src/main/java/org/apache/fluo/core/util/OracleServerUtils.java

[GitHub] kpm1985 commented on a change in pull request #1043: FLUO-1002 Create integration test for `FluoAdmin.remove()`

2018-06-22 Thread GitBox
kpm1985 commented on a change in pull request #1043: FLUO-1002 Create integration test for `FluoAdmin.remove()` URL: https://github.com/apache/fluo/pull/1043#discussion_r197361459 ## File path:

[GitHub] keith-turner removed a comment on issue #908: Discuss get jars behavior

2018-07-30 Thread GitBox
keith-turner removed a comment on issue #908: Discuss get jars behavior URL: https://github.com/apache/fluo/issues/908#issuecomment-408862569 > Could classes be loaded from HDFS directly using Accumulo's VFS classloader? Yes. One possible way to do this would be to run `fluo config

[GitHub] keith-turner commented on issue #908: Discuss get jars behavior

2018-07-30 Thread GitBox
keith-turner commented on issue #908: Discuss get jars behavior URL: https://github.com/apache/fluo/issues/908#issuecomment-408864422 > Could classes be loaded from HDFS directly using Accumulo's VFS classloader? Yes. One possible way to do this would be to run `fluo config -a ` and

[GitHub] GigabyteProductions commented on issue #908: Discuss get jars behavior

2018-07-28 Thread GitBox
GigabyteProductions commented on issue #908: Discuss get jars behavior URL: https://github.com/apache/fluo/issues/908#issuecomment-408652171 Could classes be loaded from HDFS directly using Accumulo's VFS classloader? See:

[GitHub] mikewalch opened a new pull request #228: Spark opt

2018-07-25 Thread GitBox
mikewalch opened a new pull request #228: Spark opt URL: https://github.com/apache/fluo-muchos/pull/228 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the

[GitHub] mikewalch closed pull request #228: Spark opt

2018-07-25 Thread GitBox
mikewalch closed pull request #228: Spark opt URL: https://github.com/apache/fluo-muchos/pull/228 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] mikewalch opened a new issue #229: Create 'client' service

2018-07-25 Thread GitBox
mikewalch opened a new issue #229: Create 'client' service URL: https://github.com/apache/fluo-muchos/issues/229 'client' nodes contain Hadoop and Accumulo software but nothing is run. This is an automated message from the

[GitHub] cjmctague commented on issue #466: Fluo must use same version of Guava as Twill

2018-07-31 Thread GitBox
cjmctague commented on issue #466: Fluo must use same version of Guava as Twill URL: https://github.com/apache/fluo/issues/466#issuecomment-409228633 Updating Twills version would allow us to bump Guava's version to `16.0.1` but we could not go any higher until Hadoop is updated to

[GitHub] ctubbsii commented on issue #466: Fluo must use same version of Guava as Twill

2018-07-31 Thread GitBox
ctubbsii commented on issue #466: Fluo must use same version of Guava as Twill URL: https://github.com/apache/fluo/issues/466#issuecomment-409433439 There's so many known vulnerabilities in previous versions of Hadoop (less than 2.7), that it probably makes sense to go ahead and bump all

[GitHub] mikewalch closed pull request #230: Fixes #229 - Create 'client' service

2018-08-01 Thread GitBox
mikewalch closed pull request #230: Fixes #229 - Create 'client' service URL: https://github.com/apache/fluo-muchos/pull/230 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

[GitHub] mikewalch closed issue #229: Create 'client' service

2018-08-01 Thread GitBox
mikewalch closed issue #229: Create 'client' service URL: https://github.com/apache/fluo-muchos/issues/229 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the

[GitHub] cjmctague closed pull request #1045: Fixes #1044

2018-07-27 Thread GitBox
cjmctague closed pull request #1045: Fixes #1044 URL: https://github.com/apache/fluo/pull/1045 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), the

[GitHub] keith-turner commented on issue #1046: Please add OWASP Dependency Check to the build (pom.xml)

2018-08-15 Thread GitBox
keith-turner commented on issue #1046: Please add OWASP Dependency Check to the build (pom.xml) URL: https://github.com/apache/fluo/issues/1046#issuecomment-413235478 We could possibly add a comment to the user script that fetches dependencies about checking deps for CVEs

[GitHub] ctubbsii commented on issue #188: Use conf/uno-local.conf if it exists

2018-08-14 Thread GitBox
ctubbsii commented on issue #188: Use conf/uno-local.conf if it exists URL: https://github.com/apache/fluo-uno/pull/188#issuecomment-413031982 The override pattern is just as popular, if not more so, by shell executable scripts. See `.bash*`, `.cshrc`, `.mavenrc`, everything in

[GitHub] ctubbsii commented on issue #188: Use conf/uno-local.conf if it exists

2018-08-14 Thread GitBox
ctubbsii commented on issue #188: Use conf/uno-local.conf if it exists URL: https://github.com/apache/fluo-uno/pull/188#issuecomment-413026702 For these kinds of things, I think a better pattern is: source the global/default env, then source user-specific overrides. So, what we

[GitHub] keith-turner commented on issue #188: Use conf/uno-local.conf if it exists

2018-08-14 Thread GitBox
keith-turner commented on issue #188: Use conf/uno-local.conf if it exists URL: https://github.com/apache/fluo-uno/pull/188#issuecomment-413028145 I thought about the override pattern but abandoned it after realizing I was executing shell code and not just dealing with key/value properties

[GitHub] annubiz opened a new issue #1046: Please add OWASP Dependency Check to the build (pom.xml)

2018-08-14 Thread GitBox
annubiz opened a new issue #1046: Please add OWASP Dependency Check to the build (pom.xml) URL: https://github.com/apache/fluo/issues/1046 Please add OWASP Dependency Check to the build (pom.xml). OWASP DC makes an outbound REST call to MITRE Common Vulnerabilities & Exposures (CVE) to

[GitHub] ctubbsii commented on issue #1046: Please add OWASP Dependency Check to the build (pom.xml)

2018-08-14 Thread GitBox
ctubbsii commented on issue #1046: Please add OWASP Dependency Check to the build (pom.xml) URL: https://github.com/apache/fluo/issues/1046#issuecomment-413094178 This has been on my personal TODO list for some time, and is a good idea for informational purposes. However, be aware that

[GitHub] ctubbsii edited a comment on issue #1046: Please add OWASP Dependency Check to the build (pom.xml)

2018-08-16 Thread GitBox
ctubbsii edited a comment on issue #1046: Please add OWASP Dependency Check to the build (pom.xml) URL: https://github.com/apache/fluo/issues/1046#issuecomment-413709535 Hmm, I'm not sure there's much to do here. I looked into it, and the OWASP plugin can be executed with a simple: `mvn

[GitHub] ctubbsii commented on issue #1046: Please add OWASP Dependency Check to the build (pom.xml)

2018-08-16 Thread GitBox
ctubbsii commented on issue #1046: Please add OWASP Dependency Check to the build (pom.xml) URL: https://github.com/apache/fluo/issues/1046#issuecomment-413709535 Hmm, I'm not sure there's much to do here. I looked into it, and the OWASP plugin can be executed with a simple: `mvn

[GitHub] keith-turner opened a new pull request #188: Use conf/uno-local.conf if it exists

2018-08-14 Thread GitBox
keith-turner opened a new pull request #188: Use conf/uno-local.conf if it exists URL: https://github.com/apache/fluo-uno/pull/188 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] cjmctague opened a new pull request #1047: Spelling corrections in comments/javadocs

2018-08-28 Thread GitBox
cjmctague opened a new pull request #1047: Spelling corrections in comments/javadocs URL: https://github.com/apache/fluo/pull/1047 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] cjmctague opened a new pull request #164: Spelling corrections for the website

2018-08-28 Thread GitBox
cjmctague opened a new pull request #164: Spelling corrections for the website URL: https://github.com/apache/fluo-website/pull/164 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] ctubbsii commented on a change in pull request #149: Fluo recipes updates

2018-08-28 Thread GitBox
ctubbsii commented on a change in pull request #149: Fluo recipes updates URL: https://github.com/apache/fluo-recipes/pull/149#discussion_r213485569 ## File path: modules/test/src/main/java/org/apache/fluo/recipes/test/AccumuloExportITBase.java ## @@ -39,7 +39,8 @@ /**

[GitHub] cjmctague opened a new pull request #149: Fluo recipes updates

2018-08-28 Thread GitBox
cjmctague opened a new pull request #149: Fluo recipes updates URL: https://github.com/apache/fluo-recipes/pull/149 It's broken into three different commits because they are three different small updates. - 756801b Java 8 Translation/simplification - d955ba2 Various smelling

[GitHub] cjmctague closed pull request #1047: Spelling corrections in comments/javadocs

2018-08-29 Thread GitBox
cjmctague closed pull request #1047: Spelling corrections in comments/javadocs URL: https://github.com/apache/fluo/pull/1047 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

[GitHub] cjmctague closed pull request #164: Spelling corrections for the website

2018-08-29 Thread GitBox
cjmctague closed pull request #164: Spelling corrections for the website URL: https://github.com/apache/fluo-website/pull/164 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

[GitHub] cjmctague closed pull request #149: Fluo recipes updates

2018-08-29 Thread GitBox
cjmctague closed pull request #149: Fluo recipes updates URL: https://github.com/apache/fluo-recipes/pull/149 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 commented on issue #185: Calling uno within a script fails

2018-07-19 Thread GitBox
ctubbsii commented on issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185#issuecomment-406467477 I'm inclined to call this a bug in the caller... not in the uno script. It can be worked around by avoiding the pipe in the `| while read`, which

[GitHub] keith-turner commented on issue #185: Calling uno within a script fails

2018-07-19 Thread GitBox
keith-turner commented on issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185#issuecomment-406483392 > I'm inclined to call this a bug in the caller. I am not sure if that while pattern is buggy because I am not too familiar with bash.

[GitHub] ctubbsii commented on issue #185: Calling uno within a script fails

2018-07-19 Thread GitBox
ctubbsii commented on issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185#issuecomment-406493019 So, I just realized `readarray` is a bash 4 thing, so it likely won't work on Macs, anyway; it's still possible to read stuff into an array first,

[GitHub] keith-turner commented on issue #185: Calling uno within a script fails

2018-07-19 Thread GitBox
keith-turner commented on issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185#issuecomment-406484842 I got things to work with redirecting stdout to another file descriptor. The place where I previously did the redirecting was called multiple

[GitHub] keith-turner edited a comment on issue #185: Calling uno within a script fails

2018-07-19 Thread GitBox
keith-turner edited a comment on issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185#issuecomment-406484842 I got things to work with saving stdout to another file descriptor. The place where I previously did the saving was called multiple

[GitHub] keith-turner edited a comment on issue #185: Calling uno within a script fails

2018-07-19 Thread GitBox
keith-turner edited a comment on issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185#issuecomment-406484842 I got things to work with saving stdout to another file descriptor. The place where I previously did the saving was called multiple

[GitHub] ctubbsii edited a comment on issue #185: Calling uno within a script fails

2018-07-19 Thread GitBox
ctubbsii edited a comment on issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185#issuecomment-406467477 I'm inclined to call this a bug in the caller... not in the uno script. It can be worked around by avoiding the pipe in the `| while read`,

[GitHub] keith-turner opened a new issue #185: Calling uno within a script fails

2018-07-19 Thread GitBox
keith-turner opened a new issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185 There is something about calling uno in a script like the following ... ```bash cat testClasses.txt | while read test_class; do uno setup accumulo

[GitHub] keith-turner edited a comment on issue #185: Calling uno within a script fails

2018-07-19 Thread GitBox
keith-turner edited a comment on issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185#issuecomment-406483392 > I'm inclined to call this a bug in the caller. I am not sure if that while pattern is buggy because I am not too familiar with

[GitHub] keith-turner commented on issue #185: Calling uno within a script fails

2018-07-19 Thread GitBox
keith-turner commented on issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185#issuecomment-406447905 Line 27 of setup-hadoop.sh is the following : ``` echo >&0 "Setting up Apache Hadoop at $HADOOP_PREFIX" ```

[GitHub] keith-turner edited a comment on issue #185: Calling uno within a script fails

2018-07-19 Thread GitBox
keith-turner edited a comment on issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185#issuecomment-406450191 Calling `uno setup` in the script outside the while loop works fine. I suspect it has something to do with the pipe to the while.

[GitHub] keith-turner commented on issue #185: Calling uno within a script fails

2018-07-19 Thread GitBox
keith-turner commented on issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185#issuecomment-406450191 Calling `uno setup` in the script outside the while loop works fine. I suspect it has something to do with the redirection for the loop.

[GitHub] ctubbsii commented on issue #185: Calling uno within a script fails

2018-07-19 Thread GitBox
ctubbsii commented on issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185#issuecomment-406491058 Unless rewritten, the built-in initial file descriptors, `0`, `1`, and `2`, all point to the same console device by default. You may be able to

[GitHub] keith-turner commented on issue #185: Calling uno within a script fails

2018-07-20 Thread GitBox
keith-turner commented on issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185#issuecomment-406614652 I am thinking of introducing a `printToConsole` function in the code that replaces the existing `echo >&0` commands.

[GitHub] mikewalch commented on issue #218: Metrics fails to setup without Fluo installed

2018-07-20 Thread GitBox
mikewalch commented on issue #218: Metrics fails to setup without Fluo installed URL: https://github.com/apache/fluo-muchos/issues/218#issuecomment-406696354 Fixed by #220 This is an automated message from the Apache Git

[GitHub] mikewalch closed pull request #227: Upgrade Accumulo to 1.9.2

2018-07-20 Thread GitBox
mikewalch closed pull request #227: Upgrade Accumulo to 1.9.2 URL: https://github.com/apache/fluo-muchos/pull/227 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 closed pull request #186: Adding Hadoop 3.1.0 checksum

2018-07-20 Thread GitBox
ctubbsii closed pull request #186: Adding Hadoop 3.1.0 checksum URL: https://github.com/apache/fluo-uno/pull/186 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] cjmctague commented on a change in pull request #2: [WIP] Refactor Webindex for move to Apache.

2018-07-17 Thread GitBox
cjmctague commented on a change in pull request #2: [WIP] Refactor Webindex for move to Apache. URL: https://github.com/apache/fluo-examples/pull/2#discussion_r203033374 ## File path: webindex/modules/core/pom.xml ## @@ -1,29 +1,28 @@

[GitHub] cjmctague opened a new pull request #2: [WIP] Refactor Webindex for move to Apache.

2018-07-16 Thread GitBox
cjmctague opened a new pull request #2: [WIP] Refactor Webindex for move to Apache. URL: https://github.com/apache/fluo-examples/pull/2 This is a work in progress. I've started with just Webindex and would like to get some feedback before starting the other two examples.

[GitHub] mikewalch opened a new pull request #226: Spark installation is now optional

2018-07-16 Thread GitBox
mikewalch opened a new pull request #226: Spark installation is now optional URL: https://github.com/apache/fluo-muchos/pull/226 * Users now need to specify Spark on a certain node to install * Default version of Spark is 2.2.2

[GitHub] ctubbsii commented on a change in pull request #2: [WIP] Refactor Webindex for move to Apache.

2018-07-16 Thread GitBox
ctubbsii commented on a change in pull request #2: [WIP] Refactor Webindex for move to Apache. URL: https://github.com/apache/fluo-examples/pull/2#discussion_r202853292 ## File path: webindex/modules/core/src/main/java/org/apache/fluo/webindex/core/Constants.java ## @@

[GitHub] ctubbsii commented on a change in pull request #2: [WIP] Refactor Webindex for move to Apache.

2018-07-16 Thread GitBox
ctubbsii commented on a change in pull request #2: [WIP] Refactor Webindex for move to Apache. URL: https://github.com/apache/fluo-examples/pull/2#discussion_r202853161 ## File path: webindex/modules/core/pom.xml ## @@ -1,29 +1,28 @@

[GitHub] ctubbsii commented on issue #187: fixes #185 No longer write to stdin

2018-07-23 Thread GitBox
ctubbsii commented on issue #187: fixes #185 No longer write to stdin URL: https://github.com/apache/fluo-uno/pull/187#issuecomment-407133899 I recommend running shellcheck. This is an automated message from the Apache Git

[GitHub] ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin

2018-07-23 Thread GitBox
ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin URL: https://github.com/apache/fluo-uno/pull/187#discussion_r204484249 ## File path: bin/impl/util.sh ## @@ -59,3 +59,23 @@ function run_setup_script() { shift

[GitHub] ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin

2018-07-23 Thread GitBox
ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin URL: https://github.com/apache/fluo-uno/pull/187#discussion_r204533594 ## File path: conf/uno.conf ## @@ -45,7 +45,7 @@ export

[GitHub] ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin

2018-07-23 Thread GitBox
ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin URL: https://github.com/apache/fluo-uno/pull/187#discussion_r204526850 ## File path: conf/uno.conf ## @@ -45,7 +45,7 @@ export

[GitHub] ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin

2018-07-23 Thread GitBox
ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin URL: https://github.com/apache/fluo-uno/pull/187#discussion_r204529974 ## File path: bin/impl/setup-metrics.sh ## @@ -94,14 +94,14 @@ $SED "/fluo.metrics.reporter.graphite/d"

[GitHub] ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin

2018-07-23 Thread GitBox
ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin URL: https://github.com/apache/fluo-uno/pull/187#discussion_r204533594 ## File path: conf/uno.conf ## @@ -45,7 +45,7 @@ export

[GitHub] ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin

2018-07-23 Thread GitBox
ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin URL: https://github.com/apache/fluo-uno/pull/187#discussion_r204533594 ## File path: conf/uno.conf ## @@ -45,7 +45,7 @@ export

[GitHub] keith-turner commented on a change in pull request #187: fixes #185 No longer write to stdin

2018-07-23 Thread GitBox
keith-turner commented on a change in pull request #187: fixes #185 No longer write to stdin URL: https://github.com/apache/fluo-uno/pull/187#discussion_r204529304 ## File path: conf/uno.conf ## @@ -45,7 +45,7 @@ export

[GitHub] keith-turner commented on a change in pull request #187: fixes #185 No longer write to stdin

2018-07-23 Thread GitBox
keith-turner commented on a change in pull request #187: fixes #185 No longer write to stdin URL: https://github.com/apache/fluo-uno/pull/187#discussion_r204536855 ## File path: conf/uno.conf ## @@ -45,7 +45,7 @@ export

[GitHub] keith-turner closed issue #185: Calling uno within a script fails

2018-07-23 Thread GitBox
keith-turner closed issue #185: Calling uno within a script fails URL: https://github.com/apache/fluo-uno/issues/185 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] keith-turner commented on a change in pull request #187: fixes #185 No longer write to stdin

2018-07-23 Thread GitBox
keith-turner commented on a change in pull request #187: fixes #185 No longer write to stdin URL: https://github.com/apache/fluo-uno/pull/187#discussion_r204498226 ## File path: bin/impl/util.sh ## @@ -59,3 +59,23 @@ function run_setup_script() { shift

[GitHub] mikewalch commented on a change in pull request #187: fixes #185 No longer write to stdin

2018-07-23 Thread GitBox
mikewalch commented on a change in pull request #187: fixes #185 No longer write to stdin URL: https://github.com/apache/fluo-uno/pull/187#discussion_r204461496 ## File path: bin/impl/util.sh ## @@ -59,3 +59,23 @@ function run_setup_script() { shift

[GitHub] cjmctague commented on a change in pull request #1045: Fixes #1044

2018-07-23 Thread GitBox
cjmctague commented on a change in pull request #1045: Fixes #1044 URL: https://github.com/apache/fluo/pull/1045#discussion_r204474960 ## File path: modules/core/src/main/java/org/apache/fluo/core/worker/finder/hash/TableRange.java ## @@ -104,21 +105,12 @@ public Range

[GitHub] ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin

2018-07-23 Thread GitBox
ctubbsii commented on a change in pull request #187: fixes #185 No longer write to stdin URL: https://github.com/apache/fluo-uno/pull/187#discussion_r204485527 ## File path: bin/impl/util.sh ## @@ -59,3 +59,23 @@ function run_setup_script() { shift

[GitHub] ctubbsii closed pull request #184: Update to Accumulo 1.9.2

2018-07-19 Thread GitBox
ctubbsii closed pull request #184: Update to Accumulo 1.9.2 URL: https://github.com/apache/fluo-uno/pull/184 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

[GitHub] ctubbsii opened a new pull request #184: Update to Accumulo 1.9.2

2018-07-18 Thread GitBox
ctubbsii opened a new pull request #184: Update to Accumulo 1.9.2 URL: https://github.com/apache/fluo-uno/pull/184 (this PR is pending 1.9.2-rc2 release as 1.9.2) This is an automated message from the Apache Git Service. To

[GitHub] mikewalch closed pull request #224: Updated the README with AMI changes

2018-07-13 Thread GitBox
mikewalch closed pull request #224: Updated the README with AMI changes URL: https://github.com/apache/fluo-muchos/pull/224 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

[GitHub] mikewalch commented on a change in pull request #224: Updated the README with AMI changes

2018-07-13 Thread GitBox
mikewalch commented on a change in pull request #224: Updated the README with AMI changes URL: https://github.com/apache/fluo-muchos/pull/224#discussion_r202461966 ## File path: README.md ## @@ -73,16 +70,16 @@ You can check the status of the nodes using the EC2

[GitHub] mikewalch closed pull request #190: Fixes Uno on MacOS

2018-08-30 Thread GitBox
mikewalch closed pull request #190: Fixes Uno on MacOS URL: https://github.com/apache/fluo-uno/pull/190 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] mikewalch opened a new pull request #190: Fixes Uno on MacOS

2018-08-30 Thread GitBox
mikewalch opened a new pull request #190: Fixes Uno on MacOS URL: https://github.com/apache/fluo-uno/pull/190 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use

[GitHub] mikewalch closed pull request #191: Several updates to Uno

2018-09-05 Thread GitBox
mikewalch closed pull request #191: Several updates to Uno URL: https://github.com/apache/fluo-uno/pull/191 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

<    3   4   5   6   7   8   9   10   11   12   >