From 2f894668708804050ca580663f8ef4fcd7df7151 Mon Sep 17 00:00:00 2001 From: unknown Date: Fri, 16 Feb 2001 15:46:52 -0600 Subject: [PATCH] COM_TABLE_DUMP and COM_BINLOG_DUMP are no longer erroneously logged to the slow query log. BitKeeper/etc/logging_ok: Logging to logging@openlogging.org accepted --- BitKeeper/etc/logging_ok | 1 + sql/sql_parse.cc | 13 ++++++++++--- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/BitKeeper/etc/logging_ok b/BitKeeper/etc/logging_ok index a508ba64c9c..a5ea56cdfee 100644 --- a/BitKeeper/etc/logging_ok +++ b/BitKeeper/etc/logging_ok @@ -1,2 +1,3 @@ mwagner@evoq.mwagner.org tfr@coyote.emotion.ee +jcole@tetra.spaceapes.com diff --git a/sql/sql_parse.cc b/sql/sql_parse.cc index 854a47fd9c0..27904d6501d 100644 --- a/sql/sql_parse.cc +++ b/sql/sql_parse.cc @@ -561,6 +561,9 @@ bool do_command(THD *thd) bool error=0; NET *net; enum enum_server_command command; + // commands which will always take a long time should be marked with + // this so that they will not get logged to the slow query log + bool slow_command=FALSE; DBUG_ENTER("do_command"); net= &thd->net; @@ -604,6 +607,7 @@ bool do_command(THD *thd) break; case COM_TABLE_DUMP: { + slow_command = TRUE; char* data = packet + 1; uint db_len = *data; uint tbl_len = *(data + db_len + 1); @@ -744,6 +748,7 @@ bool do_command(THD *thd) } case COM_BINLOG_DUMP: { + slow_command = TRUE; if(check_access(thd, FILE_ACL, any_db)) break; mysql_log.write(thd,command, 0); @@ -848,16 +853,18 @@ bool do_command(THD *thd) thd->proc_info="closing tables"; close_thread_tables(thd); /* Free tables */ } - thd->proc_info="cleaning up"; if (thd->fatal_error) send_error(net,0); // End of memory ? time_t start_of_query=thd->start_time; thd->end_time(); // Set start time + /* If not reading from backup and if the query took too long */ - if (!thd->user_time) + if (!slow_command && !thd->user_time) // do not log 'slow_command' queries { + thd->proc_info="logging slow query"; + if ((ulong) (thd->start_time - thd->time_after_lock) > long_query_time || ((thd->lex.options & (QUERY_NO_INDEX_USED | QUERY_NO_GOOD_INDEX_USED)) && @@ -867,7 +874,7 @@ bool do_command(THD *thd) mysql_slow_log.write(thd, thd->query, thd->query_length, start_of_query); } } - thd->proc_info="cleaning up2"; + thd->proc_info="cleaning up"; VOID(pthread_mutex_lock(&LOCK_thread_count)); // For process list thd->proc_info=0; thd->command=COM_SLEEP;