Re: [2/2] samza git commit: Yi's TopologyBuilder RB 34500

2015-06-01 Thread Navina Ramesh
Sorry about the confusion, Milinda. Thanks for reverting, Yi! On 6/1/15, 10:53 PM, "Yi Pan" wrote: >Hi, Milinda, > >That was an accidental mistake. I have reverted the check-in. I am still >working on that. Thanks! > >-Yi > >On Mon, Jun 1, 2015 at 9:34 PM, Milinda Pathirage >wrote: > >> Hi Nav

Re: [2/2] samza git commit: Yi's TopologyBuilder RB 34500

2015-06-01 Thread Yi Pan
Hi, Milinda, That was an accidental mistake. I have reverted the check-in. I am still working on that. Thanks! -Yi On Mon, Jun 1, 2015 at 9:34 PM, Milinda Pathirage wrote: > Hi Navina, > > Did we decided to push this patch to samza-sql branch. I thought Yi is > still working on this. Some Git

Re: [2/2] samza git commit: Yi's TopologyBuilder RB 34500

2015-06-01 Thread Milinda Pathirage
Hi Navina, Did we decided to push this patch to samza-sql branch. I thought Yi is still working on this. Some Git conflict related texts are still there in this commit. +<<< HEAD + * The callback object +=== + * The callback function +>>> SAMZA-552: use OperatorCallback to allow i

Re: Review Request 34746: Adding new CoordinatorStreamMessage "SetContainerHostMapping" and LocalityManager (SAMZA-618)

2015-06-01 Thread Yi Pan (Data Infrastructure)
> On May 30, 2015, 8:58 a.m., Yi Pan (Data Infrastructure) wrote: > > samza-core/src/main/java/org/apache/samza/container/LocalityManager.java, > > line 62 > > > > > > This would be invoked twice by checkpointManager a

Re: Review Request 34746: Adding new CoordinatorStreamMessage "SetContainerHostMapping" and LocalityManager (SAMZA-618)

2015-06-01 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34746/#review86125 --- Ship it! Ship It! - Yi Pan (Data Infrastructure) On May 30, 2015

Re: ProcessJobFactory parent process

2015-06-01 Thread Yi Pan
Hi, Lukas, Sounds great! Could you share more details about the issue "the logger wouldn't respect the lib/log4j.xml configuration file anymore" when you find more? Thanks! -Yi On Mon, Jun 1, 2015 at 2:17 PM, Lukas Steiblys wrote: > Hi Michael, > > So far none of our jobs have crashed yet and

Re: ProcessJobFactory parent process

2015-06-01 Thread Michael Ravits
Good information. Thanks for the update, Lukas! On Tue, Jun 2, 2015 at 12:17 AM, Lukas Steiblys wrote: > Hi Michael, > > So far none of our jobs have crashed yet and everything is working just > fine, but I can already see a major memory saving (my estimate is at least > 33% less memory is neede

Re: ProcessJobFactory parent process

2015-06-01 Thread Lukas Steiblys
Hi Michael, So far none of our jobs have crashed yet and everything is working just fine, but I can already see a major memory saving (my estimate is at least 33% less memory is needed) with ThreadJobFactory. One downside was that the logger wouldn't respect the lib/log4j.xml configuration file a

Re: Review Request 34888: SAMZA-691: messages-chosen counter without partition id

2015-06-01 Thread Yan Fang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34888/#review86044 --- samza-core/src/main/scala/org/apache/samza/system/SystemConsumersMe

Re: Samza wiki

2015-06-01 Thread Yan Fang
Hi Renato, Grant you the privilege to create pages. Should be able to create a page to track your GSoC progress. Thanks, Fang, Yan yanfang...@gmail.com On Sun, May 31, 2015 at 12:07 PM, Renato Marroquín Mogrovejo < renatoj.marroq...@gmail.com> wrote: > Hi all, > > I would like to create a wiki

Review Request 34888: SAMZA-691: messages-chosen counter without partition id

2015-06-01 Thread József Márton Jung
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34888/ --- Review request for samza. Repository: samza Description --- Fix for issu