kafka-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matthias J. Sax" <matth...@confluent.io>
Subject Re: [VOTE] KIP-233: Simplify StreamsBuilder#addGlobalStore
Date Wed, 03 Jan 2018 03:49:32 GMT
It must be two different names, as we add two processor to the topology:
a source processor that only read the data from a topic, and the actual
processor that maintains the global table.


-Matthias

On 1/2/18 7:14 PM, Ewen Cheslack-Postava wrote:
> +1 binding, seems like a nice simplification.
> 
> Regarding the source and processor name, do they actually need to be unique
> or could they use the same value? Since these use incrementing integers, it
> could be nice for debuggability/understanding to have them use the same
> name if possible instead of generating 2 separate names.
> 
> -Ewen
> 
> On Tue, Jan 2, 2018 at 9:12 AM, Guozhang Wang <wangguoz@gmail.com> wrote:
> 
>> On a side note, could you update the "compatibility and upgrade" section,
>> that when users start to make code changes to leverage the new API, what
>> kind of upgrade executions they'd need to do? I feel they need to rename
>> topics / etc.
>>
>> On Tue, Jan 2, 2018 at 9:10 AM, Guozhang Wang <wangguoz@gmail.com> wrote:
>>
>>> +1, thanks!
>>>
>>> On Wed, Dec 27, 2017 at 6:01 PM, Ted Yu <yuzhihong@gmail.com> wrote:
>>>
>>>> +1
>>>>
>>>> On Wed, Dec 27, 2017 at 12:15 PM, Bill Bejeck <bbejeck@gmail.com>
>> wrote:
>>>>
>>>>> +1
>>>>>
>>>>> On Wed, Dec 27, 2017 at 3:07 PM, Matthias J. Sax <
>> matthias@confluent.io
>>>>>
>>>>> wrote:
>>>>>
>>>>>> +1
>>>>>>
>>>>>> On 12/26/17 9:00 PM, Panuwat Anawatmongkhon wrote:
>>>>>>> Hi all,
>>>>>>> I would like to start the vote thread.
>>>>>>> This is link for the kip.
>>>>>>>
>>>>>>> https://cwiki.apache.org/confluence/display/KAFKA/KIP-233%
>>>> 3A+Simplify+
>>>>>> StreamsBuilder%23addGlobalStore
>>>>>>>
>>>>>>> Cheers
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>>
>>>
>>> --
>>> -- Guozhang
>>>
>>
>>
>>
>> --
>> -- Guozhang
>>
> 


Mime
View raw message