Re: Review Request: PIG-3268 Case statement support

2013-04-12 Thread Aniket Mokashi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10341/#review19085 --- Ship it! Neat! ANTLR magic! Just curious, dont we have a place

Re: Review Request: PIG-3268 Case statement support

2013-04-12 Thread Aniket Mokashi
Looks like we do have a place (QueryParserDriver.parse) and we are using it expand macro. So, you can do the case expansion there if you want to do it in java. On Fri, Apr 12, 2013 at 10:33 AM, Aniket Mokashi aniket...@gmail.comwrote:

Re: Review Request: PIG-3268 Case statement support

2013-04-11 Thread Cheolsoo Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10341/ --- (Updated April 12, 2013, 12:10 a.m.) Review request for pig. Description

Re: Review Request: PIG-3268 Case statement support

2013-04-11 Thread Cheolsoo Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10341/ --- (Updated April 12, 2013, 1:42 a.m.) Review request for pig. Changes ---

Re: Review Request: PIG-3268 Case statement support

2013-04-09 Thread Cheolsoo Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10341/ --- (Updated April 10, 2013, 3:10 a.m.) Review request for pig. Changes ---

Re: Review Request: PIG-3268 Case statement support

2013-04-09 Thread Cheolsoo Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10341/ --- (Updated April 10, 2013, 3:49 a.m.) Review request for pig. Changes ---

Review Request: PIG-3268 Case statement support

2013-04-08 Thread Cheolsoo Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10341/ --- Review request for pig. Description --- Add SQL CASE statement to Pig.