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

Multiple SQL Nexus fixes (#250, #251, #238, #254) #259

Merged
merged 11 commits into from
Nov 16, 2023

Conversation

PiJoCoder
Copy link
Collaborator

@PiJoCoder PiJoCoder commented Oct 12, 2023

  1. Allow the ability to minimize Cmd prompt window when Relog.exe imports Performance .BLG files into SQL Nexus

    image

  2. Allow the ability to import XEL files using local server time, rather than UTC time (default) via ReadTrace

    image

  3. Fixed "Save my options" functionality in the import window to ensure the import options were really saved. Some of the options weren't saved before even though the GUI shows the boxes are checked.

    image

    To test this last fix,

    • select "Save my options"
    • then enable one of the import options (for example minimize Cmd Window (relog.exe) )
    • Do the import.
    • Then close the Import dialog
    • Open it back up
    • Re-import and ensure the option is still working
  4. REDO_THREAD_PENDING_WORK wait type is now ignorable type, which means it will not show up in the Bottleneck Analysis report as a bar in the bar chart.

@PiJoCoder PiJoCoder changed the title Multiple SQL Nexus fixes Multiple SQL Nexus fixes (#250, #251, #238, #254) Oct 12, 2023
@asavioliMSFT
Copy link
Contributor

REDO_THREAD_PENDING_WORK wait type is now ignorable type, which means it will not show up in the Bottleneck Analysis report as a bar in the bar chart.

Currently I have one nexus loaded where a related wait is the highest one

Current nexus version

image

Version from the PR. We no longer see the REDO_THREAD_PENDING_WORK, but we still can see PARALLEL_REDO_WORKER_WAIT_WORK. I'm wondering if this one could also be ignored

image

@PiJoCoder
Copy link
Collaborator Author

PiJoCoder commented Nov 13, 2023 via email

@asavioliMSFT
Copy link
Contributor

Performed the tests below

  • select "Save my options"

  • then enable one of the import options (for example minimize Cmd Window (relog.exe) )

  • Do the import.

  • Then close the Import dialog

  • Open it back up

  • Re-import and ensure the option is still working

Initially I tested enabling the option minimize cmd during the relog.
Imported logs twice, after closing the import window. It worked fine
Then I reverted it, that is, unchecked the option to minimize the cmd during relog, since I prefer to see the window. It also worked fine twice.

asavioliMSFT
asavioliMSFT previously approved these changes Nov 15, 2023
hacitandogan
hacitandogan previously approved these changes Nov 16, 2023
Copy link
Contributor

@hacitandogan hacitandogan left a comment

Choose a reason for hiding this comment

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

Tested these changes and all looks fine.
Log capture time ;
image

Readtrace log capture time without local time adjustment;

image

With Local Server Time Option;

image

@asavioliMSFT
Copy link
Contributor

QDS_ASYNC_QUEUE

I think we can. Paul Randal agrees

https://www.sqlskills.com/help/waits/qds_async_queue/

@PiJoCoder PiJoCoder merged commit 55c649a into master Nov 16, 2023
1 check passed
@PiJoCoder PiJoCoder deleted the VariousFixes_PijoCoder branch November 16, 2023 17:29
@PiJoCoder PiJoCoder restored the VariousFixes_PijoCoder branch November 16, 2023 17:29
@PiJoCoder PiJoCoder deleted the VariousFixes_PijoCoder branch November 16, 2023 17:30
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.

3 participants