metron-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Otto Fowler <ottobackwa...@gmail.com>
Subject Re: [GitHub] incubator-metron issue #299: METRON-425 Stellar transformation fails to hand...
Date Thu, 06 Oct 2016 23:28:35 GMT
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 (git@git.apache.org) wrote:

> Github user justinleet commented on the issue:
>
> https://github.com/apache/incubator-metron/pull/299
>
> Good catch. I've been looking at this, and ended up looking into red
> herring. A further note on top of this is that (unsurprisingly in light of
> the cause) other reserved keywords tend to cause problems. Somewhat
> inconsistently, too. E.g.
> `"newStellarField" : "not"` fails, but
> `"newStellarField" : "false"` does not.
>
> I'd have to dig into it more, but I assume it's because not and the other
> comparison operators in the grammar expect to be followed by more things,
> and don't end up being string here.
>
> I'm open to suggestions on what the appropriate behavior is, but I really
> don't like that it's inconsistent on what gets rejected.
>
>
> ---
> 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 feature is enabled but not working,
> please
> contact infrastructure at infrastructure@apache.org or file a JIRA ticket
> with INFRA.
> ---
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message