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

Fillfactor indexes #204

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

lefivigh
Copy link

@lefivigh lefivigh commented Jul 14, 2020

It was included the return of fragmented indexes that have fill factor with 100%. This situation could cause impact on indexes with high volume of updates that could generate page splits.

@ghost
Copy link

ghost commented Jul 14, 2020

CLA assistant check
All CLA requirements met.

IF @ptochecks = 1
BEGIN
RAISERROR (N' |-Starting Indexes with fill factor = 100 pct', 10, 1) WITH NOWAIT
IF (SELECT COUNT(*)

Choose a reason for hiding this comment

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

What about?
IF EXISTS (SELECT TOP 1 1

Copy link
Author

Choose a reason for hiding this comment

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

Good point. I just reused the code that is currently in the stored procedure. Yes in some situations it could exists big differences in performance but in this case is just metadata information and not thousands of rows so differences in performance are not expected.

@lefivigh lefivigh closed this Jul 14, 2020
@lefivigh lefivigh reopened this Jul 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants