Re: Review Request 37026: SAMZA-727: Support for Kerberos

2016-05-24 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37026/#review134608 --- I was able to build after fixing the test class methods. Please fi

Re: Review Request 37026: SAMZA-727: Support for Kerberos

2016-05-24 Thread Yi Pan (Data Infrastructure)
> On May 24, 2016, 3:31 a.m., Yi Pan (Data Infrastructure) wrote: > > Hi, Chen, thanks for the update! One more comment: please rebase the patch > > against the latest master s.t. it can be applied cleanly. > > > > Thanks! > > Yi Pan (Data Infrastructure) wrote: > Before it is too late. I

Re: Review Request 37026: SAMZA-727: Support for Kerberos

2016-05-24 Thread Yi Pan (Data Infrastructure)
> On May 24, 2016, 3:31 a.m., Yi Pan (Data Infrastructure) wrote: > > Hi, Chen, thanks for the update! One more comment: please rebase the patch > > against the latest master s.t. it can be applied cleanly. > > > > Thanks! Before it is too late. I tried to rebase and ran the compilation and no

Review Request 47761: SAMZA-856: Optionally automatically commit based on number of processed messages

2016-05-24 Thread Branislav Cogic
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47761/ --- Review request for samza. Bugs: SAMZA-856 https://issues.apache.org/jira/br