[GitHub] metron issue #753: METRON-1183 Improve KDC Setup Instructions

2017-09-13 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/753 +1 by inspection. Thanks! ---

Re: Elasticsearch masters_also_are_datanodes doesn't work.

2017-09-13 Thread Matt Foley
That it doesn’t work in Ambari, at least to the point of writing thru to the config file (whether or not it actually works with ES), is quite mystifying. Silly question, but, are you pressing the “Save” button after changing the field? Are you perhaps not logged into Ambari with an ambari

[GitHub] metron pull request #748: METRON-1177 Stale running topologies seen post-ker...

2017-09-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/748 ---

[GitHub] metron issue #751: METRON-1180: Make Stellar Shell accept zookeeper quorum a...

2017-09-13 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/751 +1 by inspection. Nice fix ---

Assign issue to user (754: METRON-1184 , 713: METRON-1130 )

2017-09-13 Thread Ahmed Shah
Hello, I recently submitted a new PR and JIRA Issue. According to the dev guidelines, if I'm not able to assign the issue to my self I should send an email to dev@metron. Can the following be assigned to me?

[GitHub] metron pull request #754: METRON-1184 EC2 Deployment - Updating control_path...

2017-09-13 Thread as22323
GitHub user as22323 opened a pull request: https://github.com/apache/metron/pull/754 METRON-1184 EC2 Deployment - Updating control_path to accommodate for Linux ## Contributor Comments - EC2 Deployment - Updating control_path to accommodate for Linux In Metron 0.4.1-rc4, using

[GitHub] metron pull request #751: METRON-1180: Make Stellar Shell accept zookeeper q...

2017-09-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/751 ---

Re: Elasticsearch masters_also_are_datanodes doesn't work.

2017-09-13 Thread Matt Foley
Sorry, you’ve beat me. Mike, what have you found in full-dev? Does ES work correctly there? From: Laurens Vets Date: Wednesday, September 13, 2017 at 1:14 PM To: Matt Foley Cc: "dev@metron.apache.org" , Michael Miklavcic

[GitHub] metron issue #750: METRON-1179: Make STATS_ADD to take a list

2017-09-13 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/750 Ok, this is updated as per your comments, @ottobackwards ---

Re: [VOTE] Metron Release Candidate 0.4.1-RC4

2017-09-13 Thread Matt Foley
Just a reminder this vote is set to close this evening, and it looks to be currently passing. Concerns raised, but not -1’ed, include: - METRON-1173: Fix pointers to old stellar docs - Lauren’s issue with “masters_also_are_datanodes” ES config param possibly not editing correctly in Ambari -

[GitHub] metron issue #750: METRON-1179: Make STATS_ADD to take a list

2017-09-13 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/750 +1, thanks for the contribution and great work. ---

Re: Elasticsearch masters_also_are_datanodes doesn't work.

2017-09-13 Thread Matt Foley
Interesting, I had previously found that many (but not all) ES params are extensively processed by code fairly deep in the guts of ES itself (unrelated to Metron’s use of it), to allow this kind of flexibility. Thanks very much for sharing your results, Mike. Looks like we’ll continue with the

[GitHub] metron pull request #750: METRON-1179: Make STATS_ADD to take a list

2017-09-13 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/750#discussion_r138725654 --- Diff: metron-analytics/metron-statistics/src/main/java/org/apache/metron/statistics/StellarStatisticsFunctions.java --- @@ -141,8 +141,21 @@ public

Re: Elasticsearch masters_also_are_datanodes doesn't work.

2017-09-13 Thread Michael Miklavcic
Here's what I found. Data is flowing into ES indexes. I can change the "masters_also_are_datanodes" property and it is reflected in the node.data property in elasticsearch.yml. Here's my search for this property in the source code. It is properly being used int the ES mpack scripts afaik.

[GitHub] metron pull request #755: METRON-1185: Stellar REPL does not work on a kerbe...

2017-09-13 Thread cestella
GitHub user cestella opened a pull request: https://github.com/apache/metron/pull/755 METRON-1185: Stellar REPL does not work on a kerberized cluster when calling functions interacting with HBase ## Contributor Comments Because we put all the jars on the classpath from

Committing to the metron-bro-plugin-kafka repo

2017-09-13 Thread zeo...@gmail.com
So, I've been working on METRON-813 lately and I have an initial run at it ready to go here (squashed history, see a better history there

[GitHub] metron pull request #755: METRON-1185: Stellar REPL does not work on a kerbe...

2017-09-13 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/755#discussion_r138773096 --- Diff: metron-stellar/stellar-common/src/main/scripts/stellar --- @@ -28,7 +28,8 @@ elif [ -e /usr/lib/bigtop-utils/bigtop-detect-javahome ]; then

[GitHub] metron pull request #755: METRON-1185: Stellar REPL does not work on a kerbe...

2017-09-13 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/755#discussion_r138773199 --- Diff: metron-stellar/stellar-common/src/main/scripts/stellar --- @@ -28,7 +28,8 @@ elif [ -e /usr/lib/bigtop-utils/bigtop-detect-javahome ]; then

[GitHub] metron issue #755: METRON-1185: Stellar REPL does not work on a kerberized c...

2017-09-13 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/755 @ottobackwards I'd argue that it SHOULD be a temporary thing. At the moment things are broken as they are. ---

[GitHub] metron issue #755: METRON-1185: Stellar REPL does not work on a kerberized c...

2017-09-13 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/755 ok, if and when we look at stellar archetypes and extensions we'll have to keep this in mind ---

[GitHub] metron issue #755: METRON-1185: Stellar REPL does not work on a kerberized c...

2017-09-13 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/755 Ok, I just want to be completely clear, we're actually choosing *one* of our jars, the parsers jar, because it's an uber jar that has ALL of our stellar functions in Metron in it. I'm also

[GitHub] metron pull request #755: METRON-1185: Stellar REPL does not work on a kerbe...

2017-09-13 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/755#discussion_r138777894 --- Diff: metron-stellar/stellar-common/src/main/scripts/stellar --- @@ -28,7 +28,8 @@ elif [ -e /usr/lib/bigtop-utils/bigtop-detect-javahome ]; then

[GitHub] metron pull request #750: METRON-1179: Make STATS_ADD to take a list

2017-09-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/750 ---

[GitHub] metron issue #755: METRON-1185: Stellar REPL does not work on a kerberized c...

2017-09-13 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/755 Is this just a temporary thing? I think the longer term thing is to split the stellar out into their own modules, an possibly have modules split further by special dependencies.

[GitHub] metron issue #755: METRON-1185: Stellar REPL does not work on a kerberized c...

2017-09-13 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/metron/pull/755 Whatever it takes in the meantime. When this is plugin based, this all goes away. ---

Re: Elasticsearch masters_also_are_datanodes doesn't work.

2017-09-13 Thread Laurens Vets
I'll try to look further into it... Thanks for the help! On 2017-09-13 14:18, Matt Foley wrote: > Interesting, I had previously found that many (but not all) ES params are > extensively processed by code fairly deep in the guts of ES itself (unrelated > to Metron's use of it), to allow this

[GitHub] metron pull request #746: METRON-1173: Fix pointers to old stellar docs

2017-09-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/746 ---

[GitHub] metron pull request #755: METRON-1185: Stellar REPL does not work on a kerbe...

2017-09-13 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/755#discussion_r138776105 --- Diff: metron-stellar/stellar-common/src/main/scripts/stellar --- @@ -28,7 +28,8 @@ elif [ -e /usr/lib/bigtop-utils/bigtop-detect-javahome ]; then

[GitHub] metron pull request #755: METRON-1185: Stellar REPL does not work on a kerbe...

2017-09-13 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/755#discussion_r138776403 --- Diff: metron-stellar/stellar-common/src/main/scripts/stellar --- @@ -28,7 +28,8 @@ elif [ -e /usr/lib/bigtop-utils/bigtop-detect-javahome ]; then

[GitHub] metron issue #755: METRON-1185: Stellar REPL does not work on a kerberized c...

2017-09-13 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/755 So, I tested this on a kerberized multi-node cluster and full-dev. I ensured that I tried with and without the management functions jar and with/without kerberos as well. ---

Re: Elasticsearch masters_also_are_datanodes doesn't work.

2017-09-13 Thread Laurens Vets
Yes, I'm saving the changes. The orange bar pops up saying I have to restart elasticsearch, which I do. I'm logged in with the admin user, it has privileges to do everything else. I'm using a bare metal install via Ambari. Everything else seems to work through Ambari except this :) On 2017-09-13

[GitHub] metron issue #748: METRON-1177 Stale running topologies seen post-kerberizat...

2017-09-13 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/metron/pull/748 I committed another change. Everything seems to be working with this additional fix. Would like @anandsubbu to add his experiences working with this patch. This problem started with