Hi
Thanks for the comments so far. Since they mainly involve minor changes,
I'd like to start a vote for this KIP. Thanks!
https://lists.apache.org/thread/nwongf7bptvo6z1dfbo0ro63y48sxzqb

On Wed, Apr 23, 2025 at 8:27 AM Calvin Liu <ca...@confluent.io> wrote:

> Hi
> Just coming back from my vacation. Sorry for the late reply.
> DJ03: Sure, will update the KIP for the naming change.
> DJ04: Good catch, will update.
> DJ05: Yeah, we can ignore this filter when it is null or an empty string.
> DJ06: Will mention it uses re2j
>
> KT01: Sure, will do.
> KT02: Will update.
> KT03: Good idea, will mention it is an AND relation to the other filters.
> Thanks.
>
> On Tue, Apr 8, 2025 at 5:33 PM Kirk True <k...@kirktrue.pro> wrote:
>
>> Hi Calvin,
>>
>> Thanks for the KIP! This is not an area of understanding (yet!) but had a
>> few questions:
>>
>> KT01: I agree with David's DJ03. Would transactionalIdPattern or
>> transactionalIdFilter still convey the idea?
>>
>> KT02: Can we change the command line option -transactionalIdPatternFilter
>> to something more idiomatic like -transactional-id-pattern?
>>
>> KT03: Is it correct to assume there's an implicit AND for the different
>> filters? Additional filters constrain vs. broaden the results, right?
>>
>> Thanks,
>> Kirk
>>
>> On Mon, Apr 7, 2025, at 1:49 AM, David Jacot wrote:
>> > Hi Calvin,
>> >
>> > Thanks for the update.
>> >
>> > DJ03: I wonder whether we should call it `TransactionalIdPattern`.
>> Filter
>> > is a bit redundant in my opinion.
>> >
>> > DJ04: The response could now return `INVALID_REGULAR_EXPRESSION` error
>> if
>> > the regex is invalid.
>> >
>> > DJ05: Should we say that all transactions are returned if the pattern is
>> > null instead of empty? Or do we want to do it for both null and empty?
>> >
>> > DJ06: In the motivation, could you please explain that the pattern will
>> > rely on re2j like KIP-848? Or you could actually add this to the
>> "Proposed
>> > Changes" section which misses in the current KIP.
>> >
>> > Best,
>> > David
>> >
>> > On Sat, Apr 5, 2025 at 5:42 PM Calvin Liu <ca...@confluent.io.invalid>
>> > wrote:
>> >
>> > > Hi Chia-Ping,
>> > > Sure, I updated the field to be nullable.
>> > > Thanks
>> > >
>> > > On Fri, Apr 4, 2025 at 9:54 PM Chia-Ping Tsai <chia7...@gmail.com>
>> wrote:
>> > >
>> > > > > Do you mean the TransactionalIdPatternFilter in the
>> ListTransaction
>> > > > request
>> > > > can be nullable?
>> > > >
>> > > > Yes, and the null means “all transaction ids”
>> > > >
>> > > > Best,
>> > > > Chia-Ping
>> > > >
>> > > >
>> > > > > Calvin Liu <ca...@confluent.io.invalid> 於 2025年4月5日 凌晨2:35 寫道:
>> > > > >
>> > > > > Do you mean the TransactionalIdPatternFilter in the
>> ListTransaction
>> > > > request
>> > > > > can be nullable?
>> > > >
>> > >
>> >
>>
>

Reply via email to