Re: Ansible Docker fails to build latest

2017-04-21 Thread Otto Fowler
OK, the issue is the old old maven that is in the ansible image. Changing it to 3.3.9 resolved. On April 21, 2017 at 17:21:02, Otto Fowler (ottobackwa...@gmail.com) wrote: Yes. I am following the readme in there and starting the image, then just running mvn package -DskipTests On April

Re: [DISCUSS] Modify bylaws to allow speculative branches

2017-04-21 Thread Casey Stella
So, what would that look like from a practical perspective? * I presume commits would still associate to a JIRA, right? * Are you proposing changing the strategy from Review then Commit to Commit then Review for these branches? I know that we have some people who are active in the Hadoop project

Re: Ansible Docker fails to build latest

2017-04-21 Thread Casey Stella
Sorry, can you go through how you're getting to this error? I'm not super familiar with this part of the stack..is this compiling Metron inside of a docker image? On Tue, Apr 18, 2017 at 1:45 PM, Otto Fowler wrote: > Is it something to do with the relocation of

[GitHub] incubator-metron issue #542: METRON-873: Stellar string literals do not supp...

2017-04-21 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/542 @ottobackwards you're totally right. We desperately are in need of better stellar documentation: * A language reference * A set of introductory lessons in Stellar --- If your

[GitHub] incubator-metron pull request #542: METRON-873: Stellar string literals do n...

2017-04-21 Thread cestella
GitHub user cestella reopened a pull request: https://github.com/apache/incubator-metron/pull/542 METRON-873: Stellar string literals do not support quote escaping ## Contributor Comments Right now, in stellar, we cannot represent a string literal that contains `'foo'` if the

[GitHub] incubator-metron pull request #542: METRON-873: Stellar string literals do n...

2017-04-21 Thread cestella
Github user cestella closed the pull request at: https://github.com/apache/incubator-metron/pull/542 --- 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] incubator-metron issue #542: METRON-873: Stellar string literals do not supp...

2017-04-21 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/542 We need a stellar guide, more suited to learning stellar, than just documenting the facts of it. Something that gets you in the shell and gives you some exercises. --- If your

[GitHub] incubator-metron issue #542: METRON-873: Stellar string literals do not supp...

2017-04-21 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/542 ``` /code [Stellar]>>> " bar foo " bar \\\ foo ``` Shouldn't that produce 'bar \\ foo' (One less backslash)? It seems like it's not handling chained backslash

[GitHub] incubator-metron issue #542: METRON-873: Stellar string literals do not supp...

2017-04-21 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/542 We're not using SCHAR anymore. The current commit should address the backslash issue. Give it a try and let me know what you think. :) --- If your project is set up for it, you can

[GitHub] incubator-metron issue #534: METRON-861: Allow JVM args to be passed to CLI ...

2017-04-21 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/534 I could get behind that. Anyone else have other ideas? --- 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

Re: Reducing Warnings in Build

2017-04-21 Thread Otto Fowler
Until there is a use case supporting changing to other then the 80% plus case, this should not be changed. or a customer requirement etc. This would need some design and discussion time as well to map out all the implications for the complete pipeline. I think configurable charset should be

Re: Reducing Warnings in Build

2017-04-21 Thread JJ Meyer
Nick, You're not off base at all. This is exactly the push back I wanted to hear. I wasn't really sure if what I was proposing was the proper way of going about it. My understanding of charsets is limited, and I've usually just defaulted to UTF-8. That being said, right or wrong, my thought

Re: Reducing Warnings in Build

2017-04-21 Thread Ryan Merriman
I think Nick brings up some good points. Would there ever be a reason to not use UTF8 as the default from parsing a message on? All the tools we use for analytics work with UTF8 (am I wrong?). The only case I can see needing a configurable charset would be if a message coming from a sensor were

Re: Reducing Warnings in Build

2017-04-21 Thread Michael Miklavcic
Personally, I think it makes sense to have an inbound/outbound charset setting for the parsers. We should generally standardize on UTF-8 across Metron and its topologies, but accept potentially different charsets from the inbound sensors. That is to say, standardize the defaults for the things we

Re: Reducing Warnings in Build

2017-04-21 Thread Nick Allen
Per (2), I think it makes sense to make the charset configurable, but with the proposal of 3 separate settings, wouldn't things blow up horribly if the Parsers are producing UTF-8, but Enrichment is expecting UTF-16? They are not even speaking the same language, no? This makes me think that we

Re: Failure to Deploy "Quick Dev"

2017-04-21 Thread Casey Stella
I'm betting we need to regenerate the quickdev image after all the mpack changes. On Fri, Apr 21, 2017 at 10:45 AM, Otto Fowler wrote: > From lira: > > I 'think' that quickdev is actually build from full_dev, with metron > installed already. So it may be that we need a

[GitHub] incubator-metron issue #541: METRON-870: Add filtering by packet payload to ...

2017-04-21 Thread nishihatapalmer
Github user nishihatapalmer commented on the issue: https://github.com/apache/incubator-metron/pull/541 Correct, there's no NFA or DFA under the hood of the SequenceMatcher. You can create sequences using the regex syntax using the SequenceMatcherCompiler, as long as only

[GitHub] incubator-metron issue #539: METRON-867: In the event that we graduate, remo...

2017-04-21 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/539 I'm fine with looking at it Monday. I may take a quick look if there's any JIRAs from other projects we can steal as a starting template, but if not we can start running through the

[GitHub] incubator-metron issue #541: METRON-870: Add filtering by packet payload to ...

2017-04-21 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/541 Currently, I'm using the SequenceMatcher to compile a matching expression and then using a searcher to search in the byte array for that expression (code is

[GitHub] incubator-metron issue #541: METRON-870: Add filtering by packet payload to ...

2017-04-21 Thread nishihatapalmer
Github user nishihatapalmer commented on the issue: https://github.com/apache/incubator-metron/pull/541 When you say use regexes, do you mean use the regex syntax to create fixed length sequences, or do you mean use full regex functionality? Full regex exists using NFAs and DFAs,

[GitHub] incubator-metron issue #541: METRON-870: Add filtering by packet payload to ...

2017-04-21 Thread nishihatapalmer
Github user nishihatapalmer commented on the issue: https://github.com/apache/incubator-metron/pull/541 There is a slightly out of date (note to self: update this!) syntax document at:

[GitHub] incubator-metron pull request #541: METRON-870: Add filtering by packet payl...

2017-04-21 Thread cestella
Github user cestella closed the pull request at: https://github.com/apache/incubator-metron/pull/541 --- 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] incubator-metron issue #539: METRON-867: In the event that we graduate, remo...

2017-04-21 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/539 +1, I took a look through the site and didn't see anything wrong and also searched the code a bit for signs of incubation. Do we have tickets for any of the follow-on work

Re: Reducing Warnings in Build

2017-04-21 Thread Justin Leet
First off, I would absolutely love to see the warnings reduced and the quality of our code improved. I'm in favor of all four of the points, and I think it's a good start towards weeding out a lot of issues. So regarding question 1: That awkwardness comes about because org.simple.json is all

[GitHub] incubator-metron issue #541: METRON-870: Add filtering by packet payload to ...

2017-04-21 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/541 Hey, thanks for that feedback @nishihatapalmer ! I adjusted to use the suggested searcher. I did have one more question, I'm looking to document the possible regex's available for

Re: So we graduated...

2017-04-21 Thread Hakkı Hakan Akansel
Thanks to all. Congrats. On Fri, Apr 21, 2017 at 9:44 AM, Ali Nazemian wrote: > That's great! Congratulation everybody. > > On Fri, Apr 21, 2017 at 12:54 PM, Michael Miklavcic < > michael.miklav...@gmail.com> wrote: > > > Congrats all > > > > On Apr 20, 2017 8:38 PM,

Re: So we graduated...

2017-04-21 Thread Ali Nazemian
That's great! Congratulation everybody. On Fri, Apr 21, 2017 at 12:54 PM, Michael Miklavcic < michael.miklav...@gmail.com> wrote: > Congrats all > > On Apr 20, 2017 8:38 PM, "zeo...@gmail.com" wrote: > > > Well done everybody! Congrats > > > > Jon > > > > On Thu, Apr 20, 2017

[GitHub] incubator-metron pull request #540: METRON-869 Include build instructions fo...

2017-04-21 Thread anandsubbu
Github user anandsubbu closed the pull request at: https://github.com/apache/incubator-metron/pull/540 --- 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] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-21 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/538 Also @mmiklavc, please remember if the intended effect was to make the rpm build happen after everything else completed then install + pom dependencies didn't work. If you take a close