Skip to content

Profiler start session on load #21074

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.

Sign up for GitHub

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

Open
allancascante wants to merge 27 commits into
base: main
Choose a base branch
from
+878 −44
Open

Profiler start session on load #21074

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
3921de1
Profiler STS Integration, adding UI foundation, multi-session support…
Jan 19, 2026
980587f
lint error fixes
Jan 19, 2026
470bf5f
fix for eventClass mapping
Jan 20, 2026
e8fec4e
adding configuration for profiler buffer size
Jan 20, 2026
d9427f0
refactor on file structure
Jan 20, 2026
bbad598
adding tests
Jan 20, 2026
4b8b0a7
disabling grid options
Jan 20, 2026
a67042e
grid row display updates
Jan 20, 2026
63b9830
Merge remote-tracking branch 'origin' into dev/allancascante/profiler…
Jan 21, 2026
8d5dc67
added launch to object explorer menu and updated label
Jan 21, 2026
2ae5f10
linter fixes
Jan 26, 2026
d139422
Merge branch 'main' into dev/allancascante/profiler_integration
Jan 28, 2026
c217f70
Merge branch 'dev/allancascante/profiler_integration' of https://gith…
Jan 28, 2026
870259f
fixing linter errors
Jan 28, 2026
7f6269f
localization changes
Jan 28, 2026
6e27b14
PR feedback
Jan 29, 2026
115e1b2
fixing linter error
Jan 29, 2026
ec7b62d
logic to ask user to select session template and session name to crea…
Jan 30, 2026
7653dde
filter azure and on prem templates depending of the connections, addi…
Jan 30, 2026
dc73b7e
Merge branch 'dev/allancascante/profiler_start_session_on_load' into …
Jan 31, 2026
97f7837
Merge branch 'main' into dev/allancascante/profiler_start_session_on_…
Feb 5, 2026
c20cdf7
Merge remote-tracking branch 'origin' into dev/allancascante/profiler…
Feb 6, 2026
9b1ede7
pr comments
Feb 6, 2026
9ef5a7e
Merge branch 'main' into dev/allancascante/profiler_start_session_on_…
Feb 7, 2026
767d5ec
Merge branch 'main' into dev/allancascante/profiler_start_session_on_…
Feb 9, 2026
4934003
Merge remote-tracking branch 'origin' into dev/allancascante/profiler…
Feb 9, 2026
c010d25
localization updates
Feb 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions extensions/mssql/l10n/bundle.l10n.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2637,11 +2637,18 @@
"message": "Profiler session '{0}' created successfully. Starting profiling...",
"comment": ["{0} is the session name"]
},
"Profiler session '{0}' started successfully./{0} is the session name": {
"message": "Profiler session '{0}' started successfully.",
"comment": ["{0} is the session name"]
},
"Profiler ready. Select a session from the dropdown and click Start to begin profiling.": "Profiler ready. Select a session from the dropdown and click Start to begin profiling.",
"Stopping profiler session \"{0}\".../{0} is the session name": {
"message": "Stopping profiler session \"{0}\"...",
"comment": ["{0} is the session name"]
},
"Select a database for profiling (Azure SQL requires a specific database)": "Select a database for profiling (Azure SQL requires a specific database)",
"No databases found on the server. Please check your connection.": "No databases found on the server. Please check your connection.",
"Profiler is not supported on Microsoft Fabric SQL databases.": "Profiler is not supported on Microsoft Fabric SQL databases.",
"Proxy settings found, but without a protocol (e.g. http://): '{0}'. You may encounter connection issues while using the MSSQL extension./{0} is the proxy URL": {
"message": "Proxy settings found, but without a protocol (e.g. http://): '{0}'. You may encounter connection issues while using the MSSQL extension.",
"comment": ["{0} is the proxy URL"]
Expand Down
17 changes: 17 additions & 0 deletions extensions/mssql/src/constants/locConstants.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2729,6 +2729,12 @@ export class Profiler {
args: [sessionName],
comment: ["{0} is the session name"],
});
public static sessionStartedSuccessfully = (sessionName: string) =>
l10n.t({
message: "Profiler session '{0}' started successfully.",
args: [sessionName],
comment: ["{0} is the session name"],
});
public static profilerReady = l10n.t(
"Profiler ready. Select a session from the dropdown and click Start to begin profiling.",
);
Expand All @@ -2752,6 +2758,17 @@ export class Profiler {
args: [count],
comment: ["{0} is the number of events"],
});

// Database selection for Azure SQL
public static selectDatabaseForProfiler = l10n.t(
"Select a database for profiling (Azure SQL requires a specific database)",
);
public static noDatabasesFound = l10n.t(
"No databases found on the server. Please check your connection.",
Copy link

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

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

The warning text noDatabasesFound is shown when no user databases remain after filtering out system DBs, but the message says “No databases found on the server…”. This can be misleading when the server only returns system databases. Consider adjusting the string to explicitly mention “no user databases found” (or similar) to match the actual condition.

Suggested change
"No databases found on the server. Please check your connection.",
"No user databases found on the server. If this is unexpected, please check your connection or permissions.",

Copilot uses AI. Check for mistakes.
);
public static profilerNotSupportedOnFabric = l10n.t(
"Profiler is not supported on Microsoft Fabric SQL databases.",
);
}

export class Proxy {
Expand Down
Loading
Loading