-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Document hive.max-partitions-for-eager-load configuration property #14599
Document hive.max-partitions-for-eager-load configuration property #14599
Conversation
- The maximum number of partitions for a single table scan to haven loaded | ||
eagerly on the coordinator. Certain optimizations are not possible | ||
without eager loading. | ||
- 100,000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be
100000
* - ``hive.max-partitions-per-scan`` | ||
- Maximum number of partitions for a single table scan. | ||
- 100,000 | ||
- 1,000,000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1000000
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is consistent with all other properties on this table, and this is more readable considering the number of zeroes.
aac14ad
to
03101be
Compare
Tested locally |
Description
This adds docs for the config property added in #14225. Also a preliminary commit to reformat the table to a list-table, which is a functional no-op.
Non-technical explanation
Docs only.
Release notes
(x) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text: