Github user cestella commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93162319
--- Diff:
metron-analytics/metron-profiler/src/main/config/profiler.properties ---
@@ -18,21 +18,44 @@
#
#
-# Storm #
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93161910
--- Diff:
metron-analytics/metron-profiler/src/main/config/profiler.properties ---
@@ -18,21 +18,44 @@
#
#
-# Storm
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93161262
--- Diff:
metron-analytics/metron-profiler/src/main/config/profiler.properties ---
@@ -18,21 +18,44 @@
#
#
-# Storm
Github user cestella commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93161046
--- Diff: metron-analytics/metron-profiler-client/README.md ---
@@ -86,4 +71,73 @@ Returns: The profile measurements.
[12078.0, 8921.0,
Github user cestella commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93160600
--- Diff: metron-analytics/metron-profiler-client/README.md ---
@@ -86,4 +71,73 @@ Returns: The profile measurements.
[12078.0, 8921.0,
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93159217
--- Diff:
metron-analytics/metron-profiler/src/main/config/profiler.properties ---
@@ -18,21 +18,44 @@
#
#
-# Storm
Github user cestella commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93158724
--- Diff: metron-analytics/metron-profiler-client/README.md ---
@@ -86,4 +71,73 @@ Returns: The profile measurements.
[12078.0, 8921.0,
Github user cestella commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93146298
--- Diff:
metron-platform/metron-common/src/test/java/org/apache/metron/common/utils/StellarExecutorUtils.java
---
@@ -0,0 +1,75 @@
+/**
Github user cestella commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93142265
--- Diff:
metron-analytics/metron-profiler/src/main/config/profiler.properties ---
@@ -18,21 +18,44 @@
#
#
-# Storm #
Github user cestella commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93141706
--- Diff: metron-analytics/metron-profiler-client/README.md ---
@@ -86,4 +71,73 @@ Returns: The profile measurements.
[12078.0, 8921.0,
Github user merrimanr commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93130837
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/configuration/manager/ZkConfigurationManager.java
---
@@ -0,0
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93128655
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/configuration/manager/ZkConfigurationManager.java
---
@@ -0,0
Github user merrimanr commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93127834
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/configuration/manager/ZkConfigurationManager.java
---
@@ -0,0
Github user merrimanr commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/393#discussion_r93119805
--- Diff: metron-docker/kafkazk/bin/wait-for-it.sh ---
@@ -0,0 +1,161 @@
+#!/usr/bin/env bash
+# Use this script to test if a given TCP
Github user merrimanr commented on the issue:
https://github.com/apache/incubator-metron/pull/393
Just pushed out changes based on this initial round of feedback. Main
changes were moving the install steps to a Maven pom and adding more detail to
the documentation.
---
If your
Github user merrimanr commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93111809
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/configuration/manager/ZkConfigurationManager.java
---
@@ -0,0
Github user cestella commented on the issue:
https://github.com/apache/incubator-metron/pull/398
+1 by inspection; thanks jon!
---
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
GitHub user JonZeolla opened a pull request:
https://github.com/apache/incubator-metron/pull/398
METRON-631: Broken link on fastcapa README
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/JonZeolla/incubator-metron METRON-631
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-metron/pull/396
---
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
Github user kylerichardson commented on the issue:
https://github.com/apache/incubator-metron/pull/396
+1 by inspection. Thanks for fixing!
---
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
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93092757
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/configuration/manager/ZkConfigurationManager.java
---
@@ -0,0
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93075896
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/configuration/manager/ZkConfigurationManager.java
---
@@ -0,0
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/361
I have created METRON-630 for this. Please change the title of the pull
request.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user nickwallen commented on the issue:
https://github.com/apache/incubator-metron/pull/395
Yes, I think that makes sense. I will change it.
---
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
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/395
I like the change tot he StellarProcessorUtils, should we rename it to
StellarExecutorUtils?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/395#discussion_r93039627
--- Diff:
metron-platform/metron-common/src/test/java/org/apache/metron/common/utils/StellarExecutor.java
---
@@ -0,0 +1,84 @@
+package
26 matches
Mail list logo