Github user MohanDV commented on the issue:
https://github.com/apache/metron/pull/936
@nickwallen / @JonZeolla can you please merge this request.
---
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/940
That's great @cestella . Many thanks. I will run it up in the lab. No
problem.
---
Github user asfgit closed the pull request at:
https://github.com/apache/metron/pull/933
---
Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/941
Sorry I should have documented this better in the PR description. The
docker-machine ip address needs to be substituted in ElasticsearchTestUtils
temporarily if you want to run this on your local
Github user asfgit closed the pull request at:
https://github.com/apache/metron/pull/853
---
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/942
@MohanDV - It looks like some care was taken previously so that getting the
max of a list of mixed elements will just work. For example `MAX([1, 2d, 3f])
== 3f`.
Did you consider an im
Github user mmiklavc commented on a diff in the pull request:
https://github.com/apache/metron/pull/946#discussion_r172544606
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/metron_service.py
---
@@ -70,6
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/936
Thanks @MohanDV . Will merge this now.
---
Github user asfgit closed the pull request at:
https://github.com/apache/metron/pull/936
---
Github user nickwallen commented on the issue:
https://github.com/apache/metron-bro-plugin-kafka/pull/7
+1 Thanks, @JonZeolla !
---
Github user nickwallen commented on the issue:
https://github.com/apache/metron-bro-plugin-kafka/pull/6
@dcode
1. The JIRA created for this is
https://issues.apache.org/jira/browse/METRON-1469.
1. Please change the PR title to "METRON-1469: Kafka Plugin for Bro -
Co
Github user MohanDV commented on the issue:
https://github.com/apache/metron/pull/942
@nickwallen I dint consider implementation where STATS object can be passed
in list of mixed object . That requires to change the STATS object to be
'Comparable' type . Do you think it should be addr
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/948
When I got to the point of selecting the repo, I was unsure of what to type
for the Bro Plugin repo.
```
$ ./metron-pr948/dev-utilities/committer-utils/prepare-commit
...using se
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/948
Other than that little nit, it works great though. Thanks @JonZeolla
---
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/940
Ok, README is updated with the new topology diagram. Let me know if
there's anything else.
---
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/942
I would like to address the issue somehow or at least garner more community
feedback on this change.
As it stands, usage of the function is not very consistent. For example, I
can pass a
GitHub user cestella opened a pull request:
https://github.com/apache/metron/pull/949
METRON-1471: Migrate shuffle connections to local or shuffle
## Contributor Comments
Currently, we use shuffle groupings when we do not want to group by field.
We should, instead, use local or
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/942
The first point here is around semantics. I am assuming the semantics of
this would be a "max of maxes". So if I have a list of stats objects, I
compare the max of each one. Whichever max is th
Github user JonZeolla commented on the issue:
https://github.com/apache/metron/pull/948
I agree, updated the script and readme.
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/942
Maybe making this function so generic is going to necessitate it being so
complicated that it is harder to maintain etc.
---
Github user simonellistonball commented on the issue:
https://github.com/apache/metron/pull/942
I would say performance trumps complexity of functionality here.
---
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/948
+1
---
GitHub user xyztdanid4 opened a pull request:
https://github.com/apache/metron/pull/950
METRON-1470: Update jquery to version 3+
I updated jquerry dependency of the project due to security reason.
- [X] Is there a JIRA ticket associated with this PR? If not one needs to
be
Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/943
I spun this up in full dev and everything worked as expected. The
organization looks good to me and I can't find anything wrong with it. +1
---
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/940#discussion_r172595029
--- Diff:
metron-platform/metron-enrichment/src/main/java/org/apache/metron/enrichment/parallel/ParallelEnricher.java
---
@@ -0,0 +1,281 @@
+/**
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/943
What about upgrading.md?
---
Github user mmiklavc commented on the issue:
https://github.com/apache/metron/pull/943
@ottobackwards I believe the repercussions are the same as they are for the
existing MPack since the services are distinct from the MPack itself. We should
probably consider a separate test for this
Github user mmiklavc commented on the issue:
https://github.com/apache/metron/pull/943
Also, I will look at moving the Kibana templates to Metron as this is
something that will need to be done for Solr as well.
---
Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/940#discussion_r172656809
--- Diff:
metron-platform/metron-enrichment/src/main/java/org/apache/metron/enrichment/parallel/ParallelEnricher.java
---
@@ -0,0 +1,281 @@
+/**
+
Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/950
I ran this up in full dev and tested several different features. I could
not find any regressions. Thanks @xyztdanid4! +1
---
Github user asfgit closed the pull request at:
https://github.com/apache/metron/pull/950
---
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/940#discussion_r172694248
--- Diff:
metron-platform/metron-enrichment/src/main/java/org/apache/metron/enrichment/parallel/ParallelEnricher.java
---
@@ -0,0 +1,281 @@
+/**
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/940
+1 The unified topology works great.
---
Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/940#discussion_r172711543
--- Diff:
metron-platform/metron-enrichment/src/main/java/org/apache/metron/enrichment/parallel/ParallelEnricher.java
---
@@ -0,0 +1,281 @@
+/**
+
34 matches
Mail list logo