Review Request 48393: Integrate Kerberos with JC UI refactoring (part 2).

2016-06-07 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48393/ --- Review request for samza. Repository: samza Description --- Integrate Ke

Re: Review Request 48213: SAMZA-960: Make system producer thread safe

2016-06-07 Thread Xinyu Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48213/ --- (Updated June 8, 2016, 1:07 a.m.) Review request for samza, Chris Pettitt, Navi

Re: Review Request 48213: SAMZA-960: Make system producer thread safe

2016-06-07 Thread Xinyu Liu
> On June 6, 2016, 6:32 p.m., Chris Pettitt wrote: > > samza-hdfs/src/main/scala/org/apache/samza/system/hdfs/HdfsSystemProducer.scala, > > line 86 > > > > > > I think you can drop this synchronized call. Otherwise t

Re: Review Request 48352: Added java friendly api to KafkaMetrics

2016-06-07 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48352/#review136564 --- Ship it! I am assuming we won't run into problems where a metri

Re: Review Request 48352: Added java friendly api to KafkaMetrics

2016-06-07 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48352/ --- (Updated June 7, 2016, 11:59 p.m.) Review request for samza and Navina Ramesh.

Re: Review Request 47835: SAMZA-914: Initial draft for Java programming APIs on operators supporting DAGs

2016-06-07 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47835/ --- (Updated June 7, 2016, 7:42 p.m.) Review request for samza, Boris Shkolnik, Chr

Review Request 48356: RFC: Samza as a library

2016-06-07 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48356/ --- Review request for samza and Chris Pettitt. Repository: samza Description ---

Re: Review Request 48352: Added java friendly api to KafkaMetrics

2016-06-07 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48352/#review136518 --- Fix it, then Ship it! +1 ! samza-kafka/src/main/scala/org/apa

Re: Review Request 47835: SAMZA-914: Initial draft for Java programming APIs on operators supporting DAGs

2016-06-07 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47835/ --- (Updated June 7, 2016, 6:36 p.m.) Review request for samza, Boris Shkolnik, Chr

Review Request 48352: Added java friendly api to KafkaMetrics

2016-06-07 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48352/ --- Review request for samza and Navina Ramesh. Bugs: SAMZA-962 https://issues.

Re: Review Request 47835: SAMZA-914: Initial draft for Java programming APIs on operators supporting DAGs

2016-06-07 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47835/ --- (Updated June 7, 2016, 5:24 p.m.) Review request for samza, Boris Shkolnik, Chr

Re: Review Request 47835: SAMZA-914: Initial draft for Java programming APIs on operators supporting DAGs

2016-06-07 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47835/ --- (Updated June 7, 2016, 5:20 p.m.) Review request for samza, Boris Shkolnik, Chr

Re: Update all values in RocksDB

2016-06-07 Thread Yi Pan
Hi, David, Generally speaking, iterators will make a snapshot of key space of RocksDB. Hence, it associates with some memory overhead. More severe performance issue we saw before is that if you insert and delete tons of sessions in a short time period, the iterator seek function can be extremely s