-
Notifications
You must be signed in to change notification settings - Fork 878
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
Fix range of Max Result Row Count #5151
base: main
Are you sure you want to change the base?
Conversation
The minimal range value of the configurable parameter for MaxResultRowCount was incorrectly documented. The correct value is 10k instead of 100k.
Learn Build status updates of commit 2de8c5a: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
@KesemSharabi Important: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
#sign-off |
Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @KesemSharabi |
This pull request has been inactive for 14 days. If you're done making changes, don't forget to sign off. See the contributor guide for instructions. If you're still working and want to stop these notifications, apply the "keep-open" label to your PR. However, we don't advise long-running PRs due to the risk of merge conflicts. We'll begin auto-closing stale PRs in September 2021. Thank you! |
@KesemSharabi could you please sign off? |
@taiwu : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
The minimal range value of the configurable parameter for MaxResultRowCount was incorrectly documented. The correct value is 10k instead of 100k.