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

Fix range of Max Result Row Count #5151

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

taiwu
Copy link

@taiwu taiwu commented Oct 9, 2024

The minimal range value of the configurable parameter for MaxResultRowCount was incorrectly documented. The correct value is 10k instead of 100k.

The minimal range value of the configurable parameter for MaxResultRowCount was incorrectly documented.
The correct value is 10k instead of 100k.
Copy link

Learn Build status updates of commit 2de8c5a:

✅ Validation status: passed

File Status Preview URL Details
powerbi-docs/enterprise/service-admin-premium-workloads.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Jak-MS
Copy link
Contributor

Jak-MS commented Oct 9, 2024

@KesemSharabi
Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged Tracking label for the PR review team label Oct 9, 2024
@minghan
Copy link
Contributor

minghan commented Oct 10, 2024

#sign-off

Copy link
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @KesemSharabi

Copy link

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!
Power BI Docs Team
Contact Us
Contributor Guide

@taiwu
Copy link
Author

taiwu commented Oct 30, 2024

@KesemSharabi could you please sign off?

@github-actions github-actions bot removed the inactive label Oct 30, 2024
Copy link
Contributor

@taiwu : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link

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!
Power BI Docs Team
Contact Us
Contributor Guide

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.

5 participants