diff options
author | unknown <andrey@example.com> | 2006-08-17 14:22:59 +0200 |
---|---|---|
committer | unknown <andrey@example.com> | 2006-08-17 14:22:59 +0200 |
commit | 99adbd131addffeb1dc95e4c5f156dc831cbd66c (patch) | |
tree | f3888db9a52ec3b4680a4f1ede56911174eda337 /sql/event_db_repository.h | |
parent | e5a2cb50fa8febed0b6ecd537d3edc63d80e9dce (diff) | |
download | mariadb-git-99adbd131addffeb1dc95e4c5f156dc831cbd66c.tar.gz |
WL#3337 (Event scheduler new architecture)
Post-review fixes. Mostly whitespace, int-to-bool return value, fixed comments
sql/Makefile.am:
compile all submodules of Events before compiling the facade
sql/event_data_objects.cc:
- Use initialization list
- Clean whitespaces
- Shorten comments
- Fix comments
sql/event_data_objects.h:
- Fix whitespace
sql/event_db_repository.cc:
- Change return type from int to bool where only one error code is
returned.
- Don't use macros but get the maximal number of characters in a column
from the column
- Fix comments
- Make functions which has return value but it's not used - void.
sql/event_db_repository.h:
- Methods with only one error code int -> bool return value
- Remove declaration of fill_schema_events, a function that does not exist
sql/event_queue.cc:
- Use initialization lists
- Let find_n_remove_event delete the object thus making the code more robust.
The caller could forget to destruct the object. In addition, find_n_remove_element()
does not return a value.
- Move check_system_tables() to class Events
- Fix comments
sql/event_queue.h:
- Whitespace changes
- init_queue() should allow passing of THD
- check_system_tables moved to class Events
- find_n_remove_event() is now void
sql/event_scheduler.cc:
- Initialize res before use
- Remove end stop from message
sql/event_scheduler.h:
Add uninitialized state. The scheduler is in it before init_scheduler()
is called. The rationale is that otherwise state has no value before
the call. If the system tables were damaged the scheduler won't be initialized
but in Events::deinit() Event_scheduler::stop() will be called and this will
touch state, generating valgrind warning at minimum.
sql/events.cc:
- Whitespace changes
- Fix comments
- Make methods which have only one error code be bool instead of int
- Create temporarily a THD to be used for the initialization of Event_queue
- Event_queue::check_system_tables() moved to Events::check_system_tables
- is_started() is renamed to is_execution_of_events_started()
sql/events.h:
- Whitespace changes
- When a method returns only one error code it should be bool, not int
- is_started() renamed to is_execution_of_events_started()
sql/set_var.cc:
is_started() is renamed to is_execution_of_events_started()
sql/sql_db.cc:
The return code is not used, thus don't return anything and drop_schema_events()
is now void.
sql/sql_yacc.yy:
- Fix comments
- Remove unneeded initialization which is performed in lex_init()
sql/share/errmsg.txt:
New error message
sql/table.cc:
- Fix comments
- make table_check_intact() accespt const *table_def
sql/table.h:
Make table_check_intact() accespt const *table_def
Diffstat (limited to 'sql/event_db_repository.h')
-rw-r--r-- | sql/event_db_repository.h | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/sql/event_db_repository.h b/sql/event_db_repository.h index dce447a71e8..898e27fffe1 100644 --- a/sql/event_db_repository.h +++ b/sql/event_db_repository.h @@ -47,9 +47,6 @@ events_table_index_read_for_db(THD *thd, TABLE *schema_table, int events_table_scan_all(THD *thd, TABLE *schema_table, TABLE *event_table); -int -fill_schema_events(THD *thd, TABLE_LIST *tables, COND * /* cond */); - class Event_basic; class Event_parse_data; @@ -58,7 +55,7 @@ class Event_db_repository public: Event_db_repository(){} - int + bool create_event(THD *thd, Event_parse_data *parse_data, my_bool create_if_not, uint *rows_affected); @@ -70,7 +67,7 @@ public: drop_event(THD *thd, LEX_STRING db, LEX_STRING name, bool drop_if_exists, uint *rows_affected); - int + void drop_schema_events(THD *thd, LEX_STRING schema); bool @@ -79,7 +76,6 @@ public: bool load_named_event(THD *thd, LEX_STRING dbname, LEX_STRING name, Event_basic *et); - int open_event_table(THD *thd, enum thr_lock_type lock_type, TABLE **table); @@ -87,14 +83,14 @@ public: fill_schema_events(THD *thd, TABLE_LIST *tables, char *db); private: - int + void drop_events_by_field(THD *thd, enum enum_events_table_field field, LEX_STRING field_value); - int + bool index_read_for_db_for_i_s(THD *thd, TABLE *schema_table, TABLE *event_table, char *db); - int + bool table_scan_all_for_i_s(THD *thd, TABLE *schema_table, TABLE *event_table); static bool |