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

【Docs】The description of config item druid.worker.globalIngestionHeapLimitBytes in the indexer general configration is incorrect. #12265

Closed
keepmeup opened this issue Feb 17, 2022 · 5 comments

Comments

@keepmeup
Copy link

Please provide a detailed title (e.g. "Broker crashes when using TopN query with Bound filter" instead of just "Broker crashes").

Affected Version

druid-0.22.0
The Druid version where the problem was encountered.

Description

The description of config item druid.worker.globalIngestionHeapLimitBytes in the indexer general configration is incorrect.

The description of the configuration item is 60%.
image

But it's actually 1/6 in the code.
code:org.apache.druid.indexing.worker.config.WorkerConfig#globalIngestionHeapLimitBytes
image

@FrankChen021
Copy link
Member

Thank you for pointing out this. When the #8255 was submitted, it forgot the change above.
Could you open a PR to help us improve the doc?

@keepmeup
Copy link
Author

keepmeup commented Feb 28, 2022

Thank you for pointing out this. When the #8255 was submitted, it forgot the change above. Could you open a PR to help us improve the doc?

I will do it @FrankChen021

@FrankChen021
Copy link
Member

Thank you

@hevansDev
Copy link
Contributor

Fixed by #16654

@techdocsmith
Copy link
Contributor

Per @hevansDev , fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants