[GitHub] incubator-metron issue #299: METRON-425 Stellar transformation fails to hand...

2016-10-07 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/299 +1, I'm satisfied updating the docs and the unit test addresses the original concern seen. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-metron issue #299: METRON-425 Stellar transformation fails to hand...

2016-10-07 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/incubator-metron/pull/299 To validate Casey's point, I just ran a quickly modified version of your unit test with things single quotes, and things seem to line up quite nicely at that point on special chars and

[GitHub] incubator-metron issue #299: METRON-425 Stellar transformation fails to hand...

2016-10-06 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/299 So - based on `identifier_operand : (TRUE | FALSE) # LogicalConst | arithmetic_expr #ArithmeticOperands | STRING_LITERAL #

Re: [GitHub] incubator-metron issue #299: METRON-425 Stellar transformation fails to hand...

2016-10-06 Thread Otto Fowler
I think the readme needs to have a new section with the keywords and the need for escaping them explicitly called out? -- Sent with Airmail On October 6, 2016 at 18:56:08, justinleet (g...@git.apache.org) wrote: > Github user justinleet commented on the issue: > >