[PIG-394] Syntax for ?: requires parens in FOREACH

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 0.1.0
• Component/s: grunt
• Assignee: Xuefu Zhang
• Reporter: Ted Dunning
• Created: 24/Aug/08 18:28
• Updated: 05/Oct/11 00:01

Description

This fails

clean = FOREACH log {
ev = eventType eq '/rate/video'?'none':eventType;
GENERATE ev as event, 1 as cnt;
}

but this works

clean = FOREACH log {
ev = (eventType eq '/rate/video'?'none':eventType);
GENERATE ev as event, 1 as cnt;
}

The requirement for parens is bogus. Also, this fails with very misleading messages:

clean = FOREACH log {
ev = (eventType eq '/rate/video')?'none':eventType;
GENERATE ev as event, 1 as cnt;
}

I think that the parser needs to be completely revamped to avoid this sort of strangeness.

Issue Links

• depends upon: PIG-1618 Switch to new parser generator technology