[jira] [Created] (KAFKA-14125) More senses to application

2022-07-29 Thread Shyam Damodar Bodhare (Jira)
Shyam Damodar Bodhare created KAFKA-14125: - Summary: More senses to application Key: KAFKA-14125 URL: https://issues.apache.org/jira/browse/KAFKA-14125 Project: Kafka Issue Type: Impr

[jira] [Resolved] (KAFKA-14120) Produce Kafka Streams Skipped Records Metrics

2022-07-29 Thread Bruno Cadonna (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14120?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bruno Cadonna resolved KAFKA-14120. --- Resolution: Not A Problem > Produce Kafka Streams Skipped Records Metrics >

[jira] [Resolved] (KAFKA-14125) More senses to application

2022-07-29 Thread Mickael Maison (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14125?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mickael Maison resolved KAFKA-14125. Resolution: Invalid Closing as invalid. If you want to discuss microservice architectures,

[jira] [Created] (KAFKA-14126) Convert remaining DynamicBrokerReconfigurationTest tests to KRaft

2022-07-29 Thread David Arthur (Jira)
David Arthur created KAFKA-14126: Summary: Convert remaining DynamicBrokerReconfigurationTest tests to KRaft Key: KAFKA-14126 URL: https://issues.apache.org/jira/browse/KAFKA-14126 Project: Kafka

Re: [DISCUSS] KIP-844: Transactional State Stores

2022-07-29 Thread Bruno Cadonna
Hi Alex, Thanks for the updates! 1. Don't you want to deprecate StateStore#flush(). As far as I understand, commit() is the new flush(), right? If you do not deprecate it, you don't get rid of the error room you describe in your KIP by having a flush() and a commit(). 2. I would shorten Ma

Re: [VOTE] KIP-731: Record Rate Limiting for Kafka Connect

2022-07-29 Thread Sagar
Hey Ryan, Just curious what happened to this KIP? I see it never got enough votes.. I think this could be a useful feature. Thanks! Sagar. On Sat, May 8, 2021 at 1:16 PM Dongjin Lee wrote: > Thank you for the KIP. I need this feature. > > +1 (non-binding) > > Thanks, > Dongjin > > On Thu, Apr

Re: [DISCUSS] KIP-854 Separate configuration for producer ID expiry

2022-07-29 Thread Sagar
Thanks Justine, I thought -1 might be a good default setting for backward compatibility reasons. I am not too adamant on it either. Probably worth mentioning in the description that setting it to -1 would disable the feature? Other than that, LGTM! Thanks! Sagar. On Fri, Jul 29, 2022 at 8:37 AM

Re: [VOTE] KIP-731: Record Rate Limiting for Kafka Connect

2022-07-29 Thread Ryanne Dolan
Thanks Sagar, I have a POC implementation, but I haven't seen much traction here yet. Happy to pick this back up or hand it off if there is interest. Ryanne On Fri, Jul 29, 2022, 9:50 AM Sagar wrote: > Hey Ryan, > > Just curious what happened to this KIP? I see it never got enough votes.. I > t

Re: [DISCUSS] KIP-854 Separate configuration for producer ID expiry

2022-07-29 Thread Ismael Juma
+1 for having 1 day as the default and for including this change in the release notes. Ismael On Wed, Jul 27, 2022 at 9:16 AM Jason Gustafson wrote: > I don't think a major release is a requirement for a default change for > what it's worth. I do appreciate that there is a preference for not ro

[GitHub] [kafka-site] bbejeck opened a new pull request, #430: MINOR: Add clickable images to load iframe videos

2022-07-29 Thread GitBox
bbejeck opened a new pull request, #430: URL: https://github.com/apache/kafka-site/pull/430 This PR is a follow-up #427. This PR adds a clickable link that will load an `iframe`; in line with the [ASF privacy policy](https://privacy.apache.org/faq/committers.html) on embedded YouTube vid

[GitHub] [kafka-site] bbejeck commented on pull request #430: MINOR: Add clickable images to load iframe videos

2022-07-29 Thread GitBox
bbejeck commented on PR #430: URL: https://github.com/apache/kafka-site/pull/430#issuecomment-1199820791 ping @divijvaidya, @mimaison , and @vvcephei -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [kafka-site] bbejeck commented on pull request #430: MINOR: Add clickable images to load iframe videos

2022-07-29 Thread GitBox
bbejeck commented on PR #430: URL: https://github.com/apache/kafka-site/pull/430#issuecomment-1199926659 >One suggestion if it's not too much trouble, and if you agree... It might be obvious from context that that image is a button that plays a video, but it would be clearer if we superimpo

Re: [DISCUSS] KIP-854 Separate configuration for producer ID expiry

2022-07-29 Thread Justine Olshan
Hi all, I've updated the KIP to include the new default of 1 day and information about -1 in the description of the config. I wonder though if including -1 makes sense now that it is not the default value. Is there a benefit for manually setting -1 vs manually setting the value that transactional.

Re: [DISCUSS] KIP-854 Separate configuration for producer ID expiry

2022-07-29 Thread Sagar
Hey Justine, Thanks for that. In a way you are right, that setting the new config to -1 is equivalent to setting it equal to transactional.id.expiration.ms . The only reason I mentioned the -1 case is it gives the users the ability to disable this feature (which mayn't even be desirable based on w

[GitHub] [kafka-site] vvcephei commented on pull request #430: MINOR: Add clickable images to load iframe videos

2022-07-29 Thread GitBox
vvcephei commented on PR #430: URL: https://github.com/apache/kafka-site/pull/430#issuecomment-1200082801 Oh, that works! I was just thinking of editing the placeholder image itself to paste a stock play arrow on it. -- This is an automated message from the Apache Git Service. To respond