Haidong Ji
added a comment - Hi Sergei,
I'd like to work on this, if that is ok. I've checked the page here on https://mariadb.com/kb/en/performance-schema-tables/ . I think I know what to do.
Haidong
1. The kb articles don't always have the complete columns defined in the cc files. I'll keep note of this.
2. When comment has single quote, the way to escape it is to double it. Backslash escaping doesn't work.
3. I've built the binary a few times after making changes in my local branch. I see the changes correctly reflected using the 2 method Sergei mentioned above.
I'm pretty sure I can finish this task. Any tips/suggestions welcome. Thanks!
Haidong Ji
added a comment - I did some development work over the weekend:
1. The kb articles don't always have the complete columns defined in the cc files. I'll keep note of this.
2. When comment has single quote, the way to escape it is to double it. Backslash escaping doesn't work.
3. I've built the binary a few times after making changes in my local branch. I see the changes correctly reflected using the 2 method Sergei mentioned above.
I'm pretty sure I can finish this task. Any tips/suggestions welcome. Thanks!
I have finished editing the cc files for all performance schema tables listed. I've tested every one of them by building and then running `select column_name, column_comment from information_schema.columns where table_schema='performance_schema' and table_name='file_instances'`. They look good.
I've checked in the code to my forked branch here: https://github.com/haidong/server/tree/MDEV-25325 I plan to work on writing tests next. I've never done that before. So if you have suggestions/tips, that would be welcome. I may not be able to get to it until the weekend.
PS. I have noted the inconsistencies between the documentation and code files, and will report them also.
Haidong Ji
added a comment - - edited I have finished editing the cc files for all performance schema tables listed. I've tested every one of them by building and then running `select column_name, column_comment from information_schema.columns where table_schema='performance_schema' and table_name='file_instances'`. They look good.
I've checked in the code to my forked branch here: https://github.com/haidong/server/tree/MDEV-25325 I plan to work on writing tests next. I've never done that before. So if you have suggestions/tips, that would be welcome. I may not be able to get to it until the weekend.
PS. I have noted the inconsistencies between the documentation and code files, and will report them also.
The following tables don't have comments defined like others:
events_statements_summary_by_program
events_transactions_current
events_transactions_history
events_transactions_history_long
events_transactions_summary_by_account_by_event_name
events_transactions_summary_by_host_by_event_name
events_transactions_summary_by_thread_by_event_name
events_transactions_summary_by_user_by_event_name
events_transactions_summary_global_by_event_name
prepared_statements_instances
replication_applier_configuration
replication_applier_status
For "events_stages_current", "events_stages_history", and "events_stages_history_long", they all miss comment for "WORK_COMPLETED" and "WORK_ESTIMATED" columns;
For the same tables above, the ENUM of "NESTING_EVENT_TYPE" misses "TRANSACTION", I don't know if that's intentional or a mistake.
For "events_statements_current", "events_statements_history", and "events_statements_history_long", they all miss comment for the "NESTING_EVENT_LEVEL" column;
"setup_actors" misses comment for columns "ENABLED", "HISTORY"
"threads" misses comment for columns "HISTORY", "CONNECTION_TYPE", "THREAD_OS_ID"
Haidong Ji
added a comment - I have finished all coding tasks, including test cases, at least for the first go-round. I have also just submitted a PR: https://github.com/MariaDB/server/pull/1830
Documentation typo
"memory_global_by_event_name" should have been "memory_summary_global_by_event_name" on https://mariadb.com/kb/en/performance-schema-memory_global_by_event_name-table/
"LISTENw" should have been "LISTEN" on https://mariadb.com/kb/en/performance-schema-socket_summary_by_event_name-table/
The following tables don't have comments defined like others:
events_statements_summary_by_program
events_transactions_current
events_transactions_history
events_transactions_history_long
events_transactions_summary_by_account_by_event_name
events_transactions_summary_by_host_by_event_name
events_transactions_summary_by_thread_by_event_name
events_transactions_summary_by_user_by_event_name
events_transactions_summary_global_by_event_name
prepared_statements_instances
replication_applier_configuration
replication_applier_status
For "events_stages_current", "events_stages_history", and "events_stages_history_long", they all miss comment for "WORK_COMPLETED" and "WORK_ESTIMATED" columns;
For the same tables above, the ENUM of "NESTING_EVENT_TYPE" misses "TRANSACTION", I don't know if that's intentional or a mistake.
For "events_statements_current", "events_statements_history", and "events_statements_history_long", they all miss comment for the "NESTING_EVENT_LEVEL" column;
"setup_actors" misses comment for columns "ENABLED", "HISTORY"
"threads" misses comment for columns "HISTORY", "CONNECTION_TYPE", "THREAD_OS_ID"
Task
Critical
Hi Sergei,
I'd like to work on this, if that is ok. I've checked the page here on https://mariadb.com/kb/en/performance-schema-tables/ . I think I know what to do.
Haidong