Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/894#discussion_r161184707
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/ELASTICSEARCH/5.6.2/package/scripts/elastic_master.py
---
@@
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/894#discussion_r161185471
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/KIBANA/5.6.2/package/scripts/kibana_master.py
---
@@ -93,7 +
Github user anandsubbu commented on a diff in the pull request:
https://github.com/apache/metron/pull/894#discussion_r161185438
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/ELASTICSEARCH/5.6.2/package/scripts/elastic_slave.py
---
@@
GitHub user MohanDV opened a pull request:
https://github.com/apache/metron/pull/897
METRON-1395:Documentation missing for Produce a message to a Kafka topic
Rest API endpoint
## Contributor Comments
Added documentation for produce a message to a Kafka topic Rest API endpoint
GitHub user MohanDV opened a pull request:
https://github.com/apache/metron/pull/898
METRON-1398:Exclude the basic-error-controller from being added to the
swagger description
## Contributor Comments
We need to exclude the basic-error-controller and web-security-config
Github user asfgit closed the pull request at:
https://github.com/apache/metron/pull/896
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/884
Is the plan to deploy the jar file in the interpreter directory like the
other interpreters?
---
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/856#discussion_r161236049
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/common/utils/validation/StellarZookeeperBasedValidator.java
---
@@ -0,0 +
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/884
> Is the plan to deploy the jar file in the interpreter directory like the
other interpreters?
I don't know what that is going to look like. We need to do more research
to see how the se
Github user MohanDV closed the pull request at:
https://github.com/apache/metron/pull/898
---
GitHub user MohanDV reopened a pull request:
https://github.com/apache/metron/pull/898
METRON-1398:Exclude the basic-error-controller from being added to the
swagger description
## Contributor Comments
We need to exclude the basic-error-controller and web-security-conf
Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/888#discussion_r161239480
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/indexing_master.py
---
@@ -150,
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/856#discussion_r161239454
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/common/utils/validation/StellarZookeeperBasedValidator.java
---
@@ -0,
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/888#discussion_r161240092
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/indexing_master.py
---
@@
Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/856#discussion_r161240685
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/common/utils/validation/StellarZookeeperBasedValidator.java
---
@@ -0,0 +1,
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/865
__**BUMP**__
@nickwallen , @mmiklavc , @cestella : leaving a breadcrumb so you can
follow the PR to here. This is now a feature branch and is broken up as you
requested, so it is the
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/884
Do we have jiras for the next steps on this yet? I would be good if we
recored our intent, and who knows, maybe someone will want to work on it.
---
Github user mmiklavc commented on the issue:
https://github.com/apache/metron/pull/894
@anandsubbu did this ever work with CentOS 7 from Apache master? I didn't
touch that service_check.py file, and furthermore I see there are hardcoded
ports for ES.
---
Github user mmiklavc commented on the issue:
https://github.com/apache/metron/pull/894
Oh, I see what that is. This appears to be a neglected piece of code. ES
has this but Kibana does not, for instance. The metainfo.xml allows you to add
"commandScript" items that are listed in the s
Github user mmiklavc commented on the issue:
https://github.com/apache/metron/pull/894
fwiw, this service check also fails in full dev. Git history shows me as
creating the file because I moved the entire directory (5.6.2 name change) as
part of the ES upgrade, but this code hasn't be
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/884
This is really good work @nickwallen. I am +1 on this, but would like to
see jiras on the next steps entered if possible as a side note.
---
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/879
Ok, @justinleet has given a +1, do we have any existing reservations after
the discussion thread and the review here on this work? If not, then I'm going
to commit on Monday.
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/879
I'm not getting in front of the train on this.
I am neutral on this other than my comments to help the conversation and
attempt to focus things.
I don't know how to enter "don't min
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/879
haha @ottobackwards neutral would be a +0, which is fine. Thanks for your
constructive comments on the discuss thread and here. As always, they're much
appreciated. :)
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/879
+0
---
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/884
Thanks for the reviews @ottobackwards and @merrimanr.
Here are next steps for where I'd like to use this functionality.
- [Automate Install of Stellar Interpreter in
Zeppelin](h
Github user asfgit closed the pull request at:
https://github.com/apache/metron/pull/884
---
Github user mattf-horton commented on the issue:
https://github.com/apache/metron/pull/865
@ottobackwards , I'll try to refresh my review this weekend.
---
GitHub user mmiklavc opened a pull request:
https://github.com/apache/metron/pull/899
METRON-1405: Add Boyer-Moore majority vote algorithm to Stellar
## Contributor Comments
Addresses https://issues.apache.org/jira/browse/METRON-1405
A proper majority requires 2 pas
29 matches
Mail list logo
