Skip to content

KAFKA-1834: No Response when handle LeaderAndIsrRequest some case#40

Closed
tedxia wants to merge 1 commit into
apache:0.8.2from
tedxia:fix-noresponse-on-become-leader-or-follower
Closed

KAFKA-1834: No Response when handle LeaderAndIsrRequest some case#40
tedxia wants to merge 1 commit into
apache:0.8.2from
tedxia:fix-noresponse-on-become-leader-or-follower

Conversation

@tedxia

@tedxia tedxia commented Dec 29, 2014

Copy link
Copy Markdown

PR for KAFKA-1834

When a replica become leader or follower, if this broker no exist in assigned replicas, there are no response.

@joestein

Copy link
Copy Markdown
Contributor

Hi @tedxia do you mind please to use the review tool https://cwiki.apache.org/confluence/display/KAFKA/Kafka+patch+review+tool so folks can review your patch on JIRA, thanks!

@tedxia

tedxia commented Dec 30, 2014

Copy link
Copy Markdown
Author

@joestein I have add patch through kafka patch review tool, please review it, thank you.

@ijuma

ijuma commented Jul 20, 2015

Copy link
Copy Markdown
Member

@tedxia, can you please close this PR since a patch was attached to JIRA (we can't do it ourselves without asking Apache Infra via a ticket)?

ymatsuda pushed a commit to ymatsuda/kafka that referenced this pull request Sep 9, 2015
@stumped2 stumped2 closed this Feb 2, 2016
resetius added a commit to resetius/kafka that referenced this pull request Apr 22, 2016
[LOGBROKER-1394] lost+found (fixes)
satishd referenced this pull request in satishd/kafka Mar 26, 2020
Fixed a few boundary issues in RLMMWithTopicStorage
rustd pushed a commit to rustd/pranavfinaldemokafka that referenced this pull request Feb 9, 2024
…che#40)

When building streams metadata we want to build even if the host is empty as it is a common way to find the other host addresses

Reviewers: John Roesler <vvcephei@apache.org>, Anna Sophie Blee-Goldman <ableegoldman@apache.org>

Co-authored-by: Walker Carlson <18128741+wcarlson5@users.noreply.github.com>
fvaleri added a commit to fvaleri/kafka that referenced this pull request Nov 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants