[GitHub] incubator-metron pull request #268: METRON-421 Make Stellar Profiler Client ...

[cestella]

Github user cestella commented on a diff in the pull request:

https://github.com/apache/incubator-metron/pull/268#discussion_r80693264
  
--- Diff: 
metron-analytics/metron-profiler-client/src/test/resources/core-site.xml ---
@@ -0,0 +1,99 @@
+  
+
+
+  fs.defaultFS
+  hdfs://node1:8020
--- End diff --

I think this is the cause of the unit test failures.  We really shouldn't 
have a core-site.xml here.  What's caused it to be required?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-metron pull request #268: METRON-421 Make Stellar Profiler Client ...

[nickwallen]

GitHub user nickwallen opened a pull request:

https://github.com/apache/incubator-metron/pull/268

METRON-421 Make Stellar Profiler Client API Accessible in Parser and 
Enrichment Topologies

[METRON-421](https://issues.apache.org/jira/browse/METRON-421)

These changes were required to make the Stellar Profiler Client API 
accessible from the Parser and Enrichment topologies.

* Pushing the metron-profiler-client archive to the Metron host to make the 
library accessible to the Stellar Shell.
* Added the metron-profiler-client as a dependency to the Parser and 
Enrichment topologies so that the Uber jar contains the Profiler Client API.
* Fixed an issue with potentially unsafe String to Bytes conversion that 
may cause issues on different platforms.
* Added a README for the metron-profiler-client to describe its usage.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/nickwallen/incubator-metron METRON-421

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-metron/pull/268.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #268


commit 1ef0bf6670b3054eb0561061dc3f05fc56fa3123
Author: Nick Allen 
Date:   2016-09-19T21:23:44Z

METRON-421 Added profiler-client dependency to Enrichment and Parser 
topology so that Profile API can be used

commit 1a902c839007a8ddfcc26bf608a91c5f0d921cea
Author: Nick Allen 
Date:   2016-09-19T21:25:28Z

METRON-421 Need to push profiler-client archive to the deployed host

commit 11416ce40fad361fd8db52635f0952fd52fe25f1
Author: Nick Allen 
Date:   2016-09-20T21:47:34Z

METRON-421 Row key builder using unsafe string to bytes conversion

commit 8c46cd9be14c324f03f1b08f464ef727a52d66cb
Author: Nick Allen 
Date:   2016-09-21T14:22:54Z

METRON-421 Added README for the profiler client




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---