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 ([email protected]) 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 [email protected] or file a JIRA ticket > with INFRA. > --- >
