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

Tiagofe Perfmon counter translation #171

Merged

Conversation

tiagodiasfernandes
Copy link
Contributor

@tiagodiasfernandes tiagodiasfernandes commented May 24, 2022

Fixes # .

Changes proposed in this pull request:
It was created a new PowerShell method to translate the performance counter xml to local language if not in english.

How to test this code:
Execute pssdiag on a non English OS and SQL.

Has been tested on (remove any that don't apply):

  • SQL Server 2019
  • Azure SQL Virtual Machine

Translate Performance Counters from english to server local language.
Not ready to prod, 1st version.
This performance counter is missing a backslash
created translate process with hashtables - 1 minute execution improvement - takes 11 seconds now
created confirmation of translation to duplicated performance counters names
Added if statement if language is not English.
Optimized performance changing select-object logic.
Full execution now takes 5 seconds
Improved some performance .
added the new ps1 process and removed diagutil.exe call
added file to project
small bug, the "-"
Old cmdlet was not getting correct statements
@PiJoCoder PiJoCoder changed the title Tiagofe perfmon translate Tiagofe Perfmon counter translation Jun 24, 2022
@PiJoCoder
Copy link
Contributor

PiJoCoder commented Jun 29, 2022

Completed testing on a Spanish-language OS. Looks good. Will be merging

image

image

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