Re: [VOTE] KIP-622 Add currentSystemTimeMs and currentStreamTimeMs to ProcessorContext

Yet another quick FYI. While implementing KIP-820, we found that `api.MockProcessorContext` was missing these new methods as well. We added the new method to the new `api.MockProcessorContext` via https://issues.apache.org/jira/browse/KAFKA-13654. Please let us know if there are any concerns. I

Re: [VOTE] KIP-622 Add currentSystemTimeMs and currentStreamTimeMs to ProcessorContext

Just a quick FYI. KIP-622 overlapped with KIP-478. We added the new method to the new `api.ProcessorContext` via https://issues.apache.org/jira/browse/KAFKA-13699 for 3.2.0 release. Please let us know if there are any concerns. I updated the KIP accordingly. -Matthias On 3/5/21 8:42 PM, Ro

Re: [VOTE] KIP-622 Add currentSystemTimeMs and currentStreamTimeMs to ProcessorContext

Hello all, Based on the feedback of the pr https://github.com/apache/kafka/pull/9744, there are following changes done to the kip

Re: [VOTE] KIP-622 Add currentSystemTimeMs and currentStreamTimeMs to ProcessorContext

Hello all, I am closing the vote for this KIP: https://cwiki.apache.org/confluence/display/KAFKA/KIP-622%3A+Add+currentSystemTimeMs+and+currentStreamTimeMs+to+ProcessorContext Summary of the KIP: Planning to add two new methods to ProcessorContext: 1. long currentSystemTimeMs() to fetch wall-cloc

Re: [VOTE] KIP-622 Add currentSystemTimeMs and currentStreamTimeMs to ProcessorContext

Sorry for jumping into this so late, Thanks for the KIP, I'm a +1 (binding) -Bill On Sun, Jul 26, 2020 at 11:06 AM John Roesler wrote: > Thanks William, > > I’m +1 (binding) > > Thanks, > John > > On Fri, Jul 24, 2020, at 20:22, Sophie Blee-Goldman wrote: > > Thanks all, +1 (non-binding) > > >

Re: [VOTE] KIP-622 Add currentSystemTimeMs and currentStreamTimeMs to ProcessorContext

Thanks William, I’m +1 (binding) Thanks, John On Fri, Jul 24, 2020, at 20:22, Sophie Blee-Goldman wrote: > Thanks all, +1 (non-binding) > > Cheers, > Sophie > > On Wed, Jul 8, 2020 at 4:02 AM Bruno Cadonna wrote: > > > Thanks Will and Piotr, > > > > +1 (non-binding) > > > > Best, > > Bruno >

Re: [VOTE] KIP-622 Add currentSystemTimeMs and currentStreamTimeMs to ProcessorContext

Thanks all, +1 (non-binding) Cheers, Sophie On Wed, Jul 8, 2020 at 4:02 AM Bruno Cadonna wrote: > Thanks Will and Piotr, > > +1 (non-binding) > > Best, > Bruno > > On Wed, Jul 8, 2020 at 8:12 AM Matthias J. Sax wrote: > > > > Thanks for the KIP. > > > > +1 (binding) > > > > > > -Matthias > > >

Re: [VOTE] KIP-622 Add currentSystemTimeMs and currentStreamTimeMs to ProcessorContext

Thanks Will and Piotr, +1 (non-binding) Best, Bruno On Wed, Jul 8, 2020 at 8:12 AM Matthias J. Sax wrote: > > Thanks for the KIP. > > +1 (binding) > > > -Matthias > > On 7/7/20 11:48 AM, William Bottrell wrote: > > Hi everyone, > > > > I'd like to start a vote for adding two new time API's to P

Re: [VOTE] KIP-622 Add currentSystemTimeMs and currentStreamTimeMs to ProcessorContext

Thanks for the KIP. +1 (binding) -Matthias On 7/7/20 11:48 AM, William Bottrell wrote: > Hi everyone, > > I'd like to start a vote for adding two new time API's to ProcessorContext. > > Add currentSystemTimeMs and currentStreamTimeMs to ProcessorContext >

[VOTE] KIP-622 Add currentSystemTimeMs and currentStreamTimeMs to ProcessorContext

Hi everyone, I'd like to start a vote for adding two new time API's to ProcessorContext. Add currentSystemTimeMs and currentStreamTimeMs to ProcessorContext Thanks