Re: Review Request 35676: Checkpoint migration

2015-07-08 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35676/#review90976 --- @Naveen, overall looks good. I just have one point in the previous r

Re: Review Request 35676: Checkpoint migration

2015-07-08 Thread Yi Pan (Data Infrastructure)
> On July 8, 2015, 7:34 p.m., Yi Pan (Data Infrastructure) wrote: > > samza-core/src/main/scala/org/apache/samza/migration/JobRunnerMigration.scala, > > line 46 > > > > > > This reminds me of one thing: what if the j

Re: Review Request 35676: Checkpoint migration

2015-07-08 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35676/#review90966 --- nit: could you also link this RB back to the JIRA? Thanks! - Yi Pan

Re: Review Request 35676: Checkpoint migration

2015-07-08 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35676/#review90962 --- @Naveen, I had a question regarding to the migration w/ mixed inputs

Re: Review Request 35676: Checkpoint migration

2015-07-07 Thread Naveen Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35676/ --- (Updated July 8, 2015, 1:41 a.m.) Review request for samza. Repository: samza

Re: Review Request 35676: Checkpoint migration

2015-07-02 Thread Naveen Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35676/ --- (Updated July 2, 2015, 9:10 p.m.) Review request for samza. Repository: samza

Re: Review Request 35676: Checkpoint migration

2015-06-24 Thread Naveen Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35676/ --- (Updated June 24, 2015, 6:46 p.m.) Review request for samza. Repository: samz

Review Request 35676: Checkpoint migration

2015-06-19 Thread Naveen Somasundaram
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35676/ --- Review request for samza. Repository: samza Description --- SAMZA-615: C