Re: Review Request 30634: Serde for Stream Tuples

> On Feb. 5, 2015, 2:05 a.m., Yi Pan (Data Infrastructure) wrote: > > Could you link this to SAMZA-484? Sure. > On Feb. 5, 2015, 2:05 a.m., Yi Pan (Data Infrastructure) wrote: > > samza-sql/src/main/java/org/apache/samza/sql/data/string/StringData.java, > > line 29 > >

Re: Review Request 30634: Serde for Stream Tuples

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30634/#review71121 --- Could you link this to SAMZA-484? samza-sql/src/main/java/org/apac

Re: [DISCUSS] SQL workflow

Hey all, @Jakob, yeah I was thinking we'll follow our normal flow. RTC. I just wanted to set expectation that the code committed might be not up to our normal quality initially (missing docs, no tests, etc). Until the quality is raised, we should think of this module as experimental. @Milinda, aw

Re: [DISCUSS] SQL workflow

Hi Chris, Hope we no longer need the SQL API. I'll create a RB for Calcite integration. Thanks Milinda On Wed, Feb 4, 2015 at 1:31 PM, Chris Riccomini wrote: > I think so. There was some RB downtime, but it just got fixed. Yi, Navina, > Milinda, can you make sure your JIRAs have up to date RBs

Re: [DISCUSS] SQL workflow

Hi, Jacob, Yes, for sure. -Yi On Wed, Feb 4, 2015 at 11:43 AM, Jakob Homan wrote: > This submodule will still be under the review-then-commit (RTC) > regime, correct? > > On 4 February 2015 at 11:13, Yi Pan wrote: > > Just did the update w/ SAMZA-482. > > > > On Wed, Feb 4, 2015 at 10:31 AM,

Re: [DISCUSS] SQL workflow

This submodule will still be under the review-then-commit (RTC) regime, correct? On 4 February 2015 at 11:13, Yi Pan wrote: > Just did the update w/ SAMZA-482. > > On Wed, Feb 4, 2015 at 10:31 AM, Chris Riccomini > wrote: > >> I think so. There was some RB downtime, but it just got fixed. Yi, Na

Re: [DISCUSS] SQL workflow

I have updates SAMZA-484 and RB is here - https://reviews.apache.org/r/30634/diff/# Navina On 2/4/15, 11:13 AM, "Yi Pan" wrote: >Just did the update w/ SAMZA-482. > >On Wed, Feb 4, 2015 at 10:31 AM, Chris Riccomini >wrote: > >> I think so. There was some RB downtime, but it just got fixed. Yi,

Re: Review Request 30634: Serde for Stream Tuples

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30634/#review71022 --- RB contains changes applies on top of the patch from https://review

Review Request 30634: Serde for Stream Tuples

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30634/ --- Review request for samza. Repository: samza Description --- Changes retu

Re: [DISCUSS] SQL workflow

Just did the update w/ SAMZA-482. On Wed, Feb 4, 2015 at 10:31 AM, Chris Riccomini wrote: > I think so. There was some RB downtime, but it just got fixed. Yi, Navina, > Milinda, can you make sure your JIRAs have up to date RBs? > > On Wed, Feb 4, 2015 at 10:24 AM, sriram wrote: > > > Can we hav

Re: [DISCUSS] SQL workflow

I think so. There was some RB downtime, but it just got fixed. Yi, Navina, Milinda, can you make sure your JIRAs have up to date RBs? On Wed, Feb 4, 2015 at 10:24 AM, sriram wrote: > Can we have updated RBs for all the three sub tasks before we commit? This > would help us to review even after w

Re: [DISCUSS] SQL workflow

Can we have updated RBs for all the three sub tasks before we commit? This would help us to review even after we commit. On Wed, Feb 4, 2015 at 10:15 AM, Chris Riccomini wrote: > Hey all, > > Yi, Navina, and Milinda have been working on SAMZA-390 sub-tickets related > to SQL operators. We're get

Re: Instability in Samza's Git repo

Hey all, This has been fixed by ASF infra. Cheers, Chris On Mon, Feb 2, 2015 at 11:44 AM, Chris Riccomini wrote: > Hey all, > > Yi just discovered that some checksums appear to be missing from our > Apache git repo. I've opened a ticket with INFRA: > > https://issues.apache.org/jira/browse/I

[DISCUSS] SQL workflow

Hey all, Yi, Navina, and Milinda have been working on SAMZA-390 sub-tickets related to SQL operators. We're getting to the point where the amount of work floating around is quite large, and some tickets build off of others. I'm proposing that we commit this work into a samza-sql submodule on mast

Review Request 30627: REVISION: operator APIs: rebased and cleaned up

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30627/ --- Review request for samza, Chinmay Soman, Chris Riccomini, Jay Kreps, Milinda Pat

Review Request 30610: SAMZA-536

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30610/ --- Review request for samza. Bugs: SAMZA-536 https://issues.apache.org/jira/br