Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HOTFIX: bug updating cache when loading group metadata (0.9.0) #464

Closed
wants to merge 1 commit into from

Conversation

hachikuji
Copy link
Contributor

No description provided.

@hachikuji hachikuji closed this Nov 9, 2015
efeg pushed a commit to efeg/kafka that referenced this pull request Jan 29, 2020
hachikuji pushed a commit to hachikuji/kafka that referenced this pull request Dec 11, 2020
…ager listener callback (apache#464)

* KC-1184: Consume metadata log and queue for BrokerMetadataListener

* Cleanup test

* Addressing comments

* Controller: Fix BrokerRegistration + BrokerHeartbeat RPC

* Workaround for Scala 2.12 SAM type-inference bug

scala/bug#9756

* Address comments + fix BrokerLifecycleManagerTest failure

* ControllerApis: Fix BrokerRegistrationResponse lease duration

* KC-1270: Make use of the MetaLogListener callback that supplies us with new leader info

- Integrate w/ the BrokerMetadataListener/MetadataCache for the broker
to be notified through RAFT
- Remove active controller ID from the heartbeat response

* WIP: KC-1270

* Revert "WIP: KC-1270"

This reverts commit 82fff44.

* KC-1270

* Fix post-merge compilation error w/ Scala 2.12

* KC-1270: Fix BrokerMetadataListenerTest

* Fix Broker Controller discovery w/ LocalLogManager

- Configure broker/controller to use a common `metadata.log.dir`
- Fix metadata checkpoints discovery to include the `metadata.log.dir`
- Minor fix in `BrokerToControllerChannelManager` to use the correct
controller for ManualMetadataUpdate in the NetworkClient

* KC-1270: Add a test

* Address review comments: Race condition in updating controller ID and
PartitionMetadata
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.

1 participant