mLocalFile(new Path(jarPath), new Path(hdfsDestDir));
+ } catch (IOException e) {
+logger.error("Failed to copy file {} to HDFS directory {}", jarPath,
hdfsDestDir);
Review comment:
Fixed
--------
This is an automated me
:
I added check to `FluoAdminImpl.initialized()`
----
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please c
rk but can be edited.
+#
Review comment:
Should all of the logging stuff move to this new section?
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
UR
the
application specific docs in the install docs to the application docs.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment
message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
rk but can be edited.
+#
Review comment:
Fixed in bebb4932c49. Let me know what you think.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above t
e
Review comment:
With #893 I think storing the accumulo credentials in ZK is a good way to go
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above t
d be nice for the old config file to just pass it
through and for the new config file only set the app name.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
in Fluo
Recipes will need to be modified.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this
mikewalch closed pull request #883: Fixes #842 Support multiple ways for
running Fluo applications
URL: https://github.com/apache/fluo/pull/883
This is an automated message from the Apache Git Service.
To respond to the
mikewalch closed issue #842: Support multiple ways for running Fluo
Applications as downstream projects
URL: https://github.com/apache/fluo/issues/842
This is an automated message from the Apache Git Service.
To respond to
mikewalch opened a new pull request #1: Initial implementation of Apache Fluo
YARN launcher
URL: https://github.com/apache/fluo-yarn/pull/1
This is an automated message from the Apache Git Service.
To respond to the
.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With
keith-turner opened a new pull request #899: fixes #893 added ability to secure
zookeeper
URL: https://github.com/apache/fluo/pull/899
This is an automated message from the Apache Git Service.
To respond to the message
ing secret) {
setProperty(CONNECTION_ZOOKEEPER_SECRET,
verifyNotNull(CONNECTION_ZOOKEEPER_SECRET, secret));
}
```
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and u
automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
URLs.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at
from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
an be done later.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastru
asfgit closed pull request #76: remove incubator from github links
URL: https://github.com/apache/fluo-website/pull/76
This is an automated message from the Apache Git Service.
To respond to the message, please log on
keith-turner opened a new pull request #77: update links in pages/ to https
URL: https://github.com/apache/fluo-website/pull/77
This is an automated message from the Apache Git Service.
To respond to the message, please log
from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
://www.tetraconcepts.com/
Review comment:
This link redirects to http://www.pssfed.com/content/tetra
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the
://www.tetraconcepts.com/
Review comment:
Could just changed it
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about
.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git
with a prefix of fluo.app.* can easily be retrieved by a Fluo
## application running on any node in the cluster.
#fluo.app.key1=val1
```
This is an automated message from the Apache Git Service.
To respond to the message
Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
cjmctague opened a new pull request #902: fixes #894 merge SimpleConfiguration
Objects
URL: https://github.com/apache/fluo/pull/902
This is an automated message from the Apache Git Service.
To respond to the message
().
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards
s are merged if they are set in multiple
SimpleConfigurations. Also does first configuration parameter listed take
precedent or does last?
This is an automated message from the Apache Git Service.
To respond to the message, please l
if a property is already set.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please co
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
alues.`
That's why I used add properties because it was more of a merge by adding
all the properties together. I could be misunderstanding it.
This is an automated message from the Apache Git Service.
To respond to the message,
precedent
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
kes precedent
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
test.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
ty()`
overwrites the value of anything with the same key.
Note how we lose `value1` in this example. Is that acceptable? In my mind
that's not really a merge but rather a rewrite of the property.
This is an automated
ty()`
overwrites the value of anything with the same key.
Note how we lose `value1` in this example. Is that acceptable? In my mind
that's not really a merge but rather a rewrite of the property if they have the
same key value.
keith-turner opened a new pull request #903: Fluo 897
URL: https://github.com/apache/fluo/pull/903
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL
n s.p
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apach
Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
keith-turner closed pull request #903: Fluo 897
URL: https://github.com/apache/fluo/pull/903
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to
hanks
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
A
keith-turner opened a new pull request #904: Fixes #897 added context to
application docs and moved some docs from?
URL: https://github.com/apache/fluo/pull/904
? install
This is an automated message from the Apache Git
liest.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
A
body of the log message.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please co
@@
+
Review comment:
Can you provide a bit of context for this? I don't understand these changes.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above
ssage from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
-
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
-
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
@@
+
Review comment:
> It had to be moved in order to work with the newer version of Checkstyle.
Where did the version of Checkstyle change?
This is an automated message from the Apache Git Service.
To respond to the mess
lugin because the rules won't work.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this servi
ore informative.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastruct
@@
+
Review comment:
I thought that's basically what my log message said, but I can reword it if
it's not clear.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and u
is is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
---
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
is nice.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Ap
asfgit closed pull request #891: Migrate to checkstyle 8.1 rules
URL: https://github.com/apache/fluo/pull/891
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use
:
It seems like this could be dropped
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this servi
the version of Fluo YARN launcher"
+ echo " "
+ exit 1
+}
+
+function build_bundled_jar {
+ app_dir=$lib/apps/$app
+ mkdir -p "$app_dir/lib"
+
+ java org.apache.fluo.command.FluoGetJars "$FLUO_CONN_PROPS" "$app"
"$app_dir/lib"
Review comment:
What hap
's alright.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
`
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
below):`
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at
automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
)`
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us
` instead of `instance`
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact
know when to trigger observers.
6. Start your Fluo instance so that your Fluo workers load the new observer.
Review comment:
Could be `Start your Fluo application so that`
This is an automated message from the Apache Git Service
mikewalch commented on issue #900: Document properties in FluoConfiguration
javadoc
URL: https://github.com/apache/fluo/issues/900#issuecomment-320965130
Thanks @kpm1985!
This is an automated message from the Apache Git
cs/index.html
```
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@inf
all yours.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
g. What
where you thinking it would hang off of?
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries ab
mment:
`conjoinTo(SimpleConfiguration)`
Means "join; combine"
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific
://www.tetraconcepts.com/
Review comment:
@cjnolet do you have any opinion on this?
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific
njoin' is 'with'
(mutual relationship) instead of 'to' (directional relationship), as in
'conjoinWith' instead of 'conjoinTo'.
----
This is an automated message from the Apache Git Service.
To respo
mikewalch closed issue #847: Create FluoGetJars command
URL: https://github.com/apache/fluo/issues/847
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
mikewalch closed issue #846: Retrieve FluoConfiguration from Zookeeper using
app name & connection.properties
URL: https://github.com/apache/fluo/issues/846
This is an automated message from the Apache Git Service
ated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
mikewalch commented on issue #847: Create FluoGetJars command
URL: https://github.com/apache/fluo/issues/847#issuecomment-321078965
Closed by #883
This is an automated message from the Apache Git Service.
To respond to the
mikewalch commented on issue #850: Create FluoScan command
URL: https://github.com/apache/fluo/issues/850#issuecomment-321079099
Closed by #883
This is an automated message from the Apache Git Service.
To respond to the
mikewalch commented on issue #849: Create FluoWorker command
URL: https://github.com/apache/fluo/issues/849#issuecomment-321079065
Closed by #883
This is an automated message from the Apache Git Service.
To respond to the
mikewalch commented on issue #848: Create FluoOracle command
URL: https://github.com/apache/fluo/issues/848#issuecomment-321079011
Closed by #883
This is an automated message from the Apache Git Service.
To respond to the
mikewalch commented on issue #851: Create FluoList command
URL: https://github.com/apache/fluo/issues/851#issuecomment-321079167
Closed by #883
This is an automated message from the Apache Git Service.
To respond to the
mikewalch closed issue #850: Create FluoScan command
URL: https://github.com/apache/fluo/issues/850
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL
mikewalch commented on issue #845: Create FluoInit command
URL: https://github.com/apache/fluo/issues/845#issuecomment-321079261
Closed by #883
This is an automated message from the Apache Git Service.
To respond to the
mikewalch closed issue #845: Create FluoInit command
URL: https://github.com/apache/fluo/issues/845
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL
mikewalch closed issue #851: Create FluoList command
URL: https://github.com/apache/fluo/issues/851
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL
mikewalch closed issue #848: Create FluoOracle command
URL: https://github.com/apache/fluo/issues/848
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL
mikewalch closed issue #849: Create FluoWorker command
URL: https://github.com/apache/fluo/issues/849
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL
mikewalch closed issue #852: Create FluoWait command
URL: https://github.com/apache/fluo/issues/852
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL
mikewalch commented on issue #852: Create FluoWait command
URL: https://github.com/apache/fluo/issues/852#issuecomment-321079299
Closed by #883
This is an automated message from the Apache Git Service.
To respond to the
mikewalch closed issue #843: Rename FluoConfiguration properties
URL: https://github.com/apache/fluo/issues/843
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and
mikewalch commented on issue #843: Rename FluoConfiguration properties
URL: https://github.com/apache/fluo/issues/843#issuecomment-321079476
Closed by #883
This is an automated message from the Apache Git Service.
To
mikewalch opened a new pull request #78: Remove incubator references on website
URL: https://github.com/apache/fluo-website/pull/78
This is an automated message from the Apache Git Service.
To respond to the message, please
mikewalch closed pull request #78: Remove incubator references on website
URL: https://github.com/apache/fluo-website/pull/78
This is an automated message from the Apache Git Service.
To respond to the message, please log
API checks in Fluo projects.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
ng fun and finishing this
before I resume school.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about th
401 - 500 of 579 matches
Mail list logo
