-
Notifications
You must be signed in to change notification settings - Fork 45
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
[QPROF] Updating session params #1283
Conversation
session_params_current allows to get the current params.
@@ -239,6 +241,9 @@ class QueryProfiler: | |||
tables_dtypes: list | |||
Datatypes of all the loaded | |||
performance tables. | |||
session_params: list |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
session_params: list | |
session_params_non_default: list |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You will need to change the name everywhere if you decide to accept this.
@@ -225,6 +225,8 @@ class QueryProfiler: | |||
Current Transaction ID. | |||
statement_id: int | |||
Current Statement ID. | |||
session_params_current: dict | |||
Current Session Parameters. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Current Session Parameters. | |
Non Default Session Parameters used | |
to run the current/active query. |
@@ -225,6 +225,8 @@ class QueryProfiler: | |||
Current Transaction ID. | |||
statement_id: int | |||
Current Statement ID. | |||
session_params_current: dict |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
session_params_current: dict | |
session_params_non_default_current: dict |
@@ -239,6 +241,9 @@ class QueryProfiler: | |||
tables_dtypes: list | |||
Datatypes of all the loaded | |||
performance tables. | |||
session_params: list | |||
Non Default Session Parameters used |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Non Default Session Parameters used | |
List of Non Default Session Parameters used |
@@ -239,6 +241,9 @@ class QueryProfiler: | |||
tables_dtypes: list | |||
Datatypes of all the loaded | |||
performance tables. | |||
session_params: list | |||
Non Default Session Parameters used | |||
to run the transactions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to run the transactions. | |
to run all the queries that are profiled. |
Should close: