Skip to content

KAFKA-2764: Change use of Properties in Copycat to Maps.#444

Closed
ewencp wants to merge 1 commit intoapache:trunkfrom
ewencp:kafka-2764-maps-not-properties
Closed

KAFKA-2764: Change use of Properties in Copycat to Maps.#444
ewencp wants to merge 1 commit intoapache:trunkfrom
ewencp:kafka-2764-maps-not-properties

Conversation

@ewencp
Copy link
Copy Markdown
Contributor

@ewencp ewencp commented Nov 6, 2015

No description provided.

@ewencp
Copy link
Copy Markdown
Contributor Author

ewencp commented Nov 6, 2015

@gwenshap or @guozhangwang This one is a trivial conversion, should be an easy review.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice :)

@guozhangwang
Copy link
Copy Markdown
Contributor

LGTM. Thanks!

@asfgit asfgit closed this in c006c59 Nov 6, 2015
hachikuji added a commit to hachikuji/kafka that referenced this pull request Nov 4, 2020
)

Test cases in `kafka.metrics.MetricsTest` were failing because of the name change from `KafkaServer` to `KafkaBroker`. This patch introduces an alternate registration path for these metrics.
wyuka pushed a commit to wyuka/kafka that referenced this pull request Mar 22, 2023
…che#444)

This should be a direct fix-up to the commit
- 28cca3e Cancellation of reassignments allowed if the resulting ISR is at min-ISR (apache#442)
davide-armand pushed a commit to aiven/kafka that referenced this pull request Dec 1, 2025
* feat(inkless): support for reloading gcs credentials

GCS credentials are automatically reloaded if gcs.credentials.path is
used to define the credentials in a separate json file.

* chore(inkless): add thread name for S3 credential watcher
jeqo pushed a commit to aiven/kafka that referenced this pull request Jan 16, 2026
* feat(inkless): support for reloading gcs credentials

GCS credentials are automatically reloaded if gcs.credentials.path is
used to define the credentials in a separate json file.

* chore(inkless): add thread name for S3 credential watcher
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.

3 participants