Re: Review Request 29754: StreamSQL operator API draft

2015-01-30 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29754/ --- (Updated Jan. 31, 2015, 7:49 a.m.) Review request for samza, Chris Riccomini, J

Re: Review Request 29754: StreamSQL operator API draft

2015-01-30 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29754/ --- (Updated Jan. 31, 2015, 7:47 a.m.) Review request for samza, Chris Riccomini, J

Re: Review Request 29754: StreamSQL operator API draft

2015-01-30 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29754/ --- (Updated Jan. 31, 2015, 2:02 a.m.) Review request for samza, Chris Riccomini, J

Re: Review Request 29754: StreamSQL operator API draft

2015-01-30 Thread Chris Riccomini
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29754/#review70380 --- Two high-level comments: * RandomTask is complex. Developers will n

Re: Review Request 29754: StreamSQL operator API draft

2015-01-30 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29754/ --- (Updated Jan. 30, 2015, 8:34 p.m.) Review request for samza, Chris Riccomini, J

Re: Review Request 29754: StreamSQL operator API draft

2015-01-30 Thread Yi Pan (Data Infrastructure)
> On Jan. 30, 2015, 6:04 p.m., Chris Riccomini wrote: > > build.gradle, line 265 > > > > > > Remove -d64 here. Tests should work on 32 bit machines. Sure. I will update this. > On Jan. 30, 2015, 6:04 p.m., Chris Ric

Re: Review Request 29754: StreamSQL operator API draft

2015-01-29 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29754/ --- (Updated Jan. 29, 2015, 7:32 p.m.) Review request for samza, Chris Riccomini, J