Hi Lucia, One question:
1. Since the proposed implementation change for withRange() method uses Optional.ofNullable() (which only catches nulls and not blank/whitespace strings), wouldn’t users still need to have code like that in the example? 2. Why don't users create RangeQuery objects that use Optional directly? What’s the benefit of introducing what appears to be a very thin utility facade? Thanks, Kirk > On Jun 21, 2023, at 9:51 AM, Kirk True <k...@kirktrue.pro> wrote: > > Hi Lucia, > > Thanks for the KIP! > > The KIP wasn’t in the email and I didn’t see it on the main KIP directory. > Here it is: > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-941%3A+Range+queries+to+accept+null+lower+and+upper+bounds > > Can the KIP be added to the main KIP page > (https://cwiki.apache.org/confluence/display/KAFKA/Kafka+Improvement+Proposals)? > That will help with discoverability and encourage discussion. > > Thanks, > Kirk > >> On Jun 15, 2023, at 2:13 PM, Lucia Cerchie <lcerc...@confluent.io.INVALID> >> wrote: >> >> Hi everyone, >> >> I'd like to discuss KIP-941, which will change the behavior of range >> queries to make it easier for users to execute full range scans when using >> interactive queries with upper and lower bounds from query parameters in >> web client requests. >> >> I much appreciate your input! >> >> Lucia Cerchie >> -- >> >> [image: Confluent] <https://www.confluent.io> >> Lucia Cerchie >> Developer Advocate >> Follow us: [image: Blog] >> <https://www.confluent.io/blog?utm_source=footer&utm_medium=email&utm_campaign=ch.email-signature_type.community_content.blog>[image: >> Twitter] <https://twitter.com/ConfluentInc>[image: Slack] >> <https://slackpass.io/confluentcommunity>[image: YouTube] >> <https://youtube.com/confluent> >> >> [image: Try Confluent Cloud for Free] >> <https://www.confluent.io/get-started?utm_campaign=tm.fm-apac_cd.inbound&utm_source=gmail&utm_medium=organic> >
