Skip to content

document DynamicConfigProvider for kafka consumer properties#10658

Merged
himanshug merged 4 commits intoapache:masterfrom
himanshug:document_dynamic_config_provider
Dec 10, 2020
Merged

document DynamicConfigProvider for kafka consumer properties#10658
himanshug merged 4 commits intoapache:masterfrom
himanshug:document_dynamic_config_provider

Conversation

@himanshug
Copy link
Copy Markdown
Contributor

documentation for #10309

Comment thread docs/development/extensions-core/kafka-ingestion.md Outdated
Comment thread docs/development/extensions-core/kafka-ingestion.md Outdated
@himanshug
Copy link
Copy Markdown
Contributor Author

thanks @jihoonson for quick review, applied both changes.

Copy link
Copy Markdown
Contributor

@jihoonson jihoonson left a comment

Choose a reason for hiding this comment

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

@himanshug thanks for quick fix. +1 after CI.

@himanshug himanshug merged commit be01976 into apache:master Dec 10, 2020
@jihoonson jihoonson added this to the 0.21.0 milestone Jan 4, 2021
JulianJaffePinterest pushed a commit to JulianJaffePinterest/druid that referenced this pull request Jan 22, 2021
…10658)

* document DynamicConfigProvider for kafka consumer properties

* Update docs/development/extensions-core/kafka-ingestion.md

Co-authored-by: Jihoon Son <jihoonson@apache.org>

* Update docs/development/extensions-core/kafka-ingestion.md

* fix doc build

Co-authored-by: Jihoon Son <jihoonson@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants