Skip to content

Commit

Permalink
Merge branch 'merge-perfschema-5.6' into 10.0
Browse files Browse the repository at this point in the history
  • Loading branch information
vuvova committed Oct 9, 2015
2 parents 01be663 + 1b41eed commit 2ca4141
Show file tree
Hide file tree
Showing 5 changed files with 75 additions and 3 deletions.
15 changes: 15 additions & 0 deletions mysql-test/suite/perfschema/r/misc.result
Expand Up @@ -92,3 +92,18 @@ object_schema object_name index_name count_fetch count_insert count_update count
test t_60905 i 2 0 0 1
test t_60905 NULL 5 5 0 1
DROP TABLE t_60905;
use test;
truncate performance_schema.events_statements_history;
truncate performance_schema.events_statements_history_long;
select * from t1;
ERROR 42S02: Table 'test.t1' doesn't exist

select mysql_errno, returned_sqlstate, message_text, errors, warnings
from performance_schema.events_statements_history where errors > 0;
mysql_errno returned_sqlstate message_text errors warnings
1146 42S02 Table 'test.t1' doesn't exist 1 0

select mysql_errno, returned_sqlstate, message_text, errors, warnings from
performance_schema.events_statements_history_long where errors > 0;
mysql_errno returned_sqlstate message_text errors warnings
1146 42S02 Table 'test.t1' doesn't exist 1 0
20 changes: 20 additions & 0 deletions mysql-test/suite/perfschema/t/misc.test
Expand Up @@ -169,3 +169,23 @@ SELECT object_schema,

DROP TABLE t_60905;


#
# Bug#11929832 - EVENTS_STATEMENTS_HISTORY HAS ERRORS=0 WHEN THERE ARE ERRORS
#
# Verify that SQL errors are properly counted.

use test;
truncate performance_schema.events_statements_history;
truncate performance_schema.events_statements_history_long;

--error ER_NO_SUCH_TABLE
select * from t1;

--echo
select mysql_errno, returned_sqlstate, message_text, errors, warnings
from performance_schema.events_statements_history where errors > 0;

--echo
select mysql_errno, returned_sqlstate, message_text, errors, warnings from
performance_schema.events_statements_history_long where errors > 0;
2 changes: 1 addition & 1 deletion storage/perfschema/ha_perfschema.cc
Expand Up @@ -205,7 +205,7 @@ maria_declare_plugin(perfschema)
0x0001,
pfs_status_vars,
NULL,
"5.6.26",
"5.6.27",
MariaDB_PLUGIN_MATURITY_STABLE
}
maria_declare_plugin_end;
Expand Down
1 change: 1 addition & 0 deletions storage/perfschema/pfs.cc
Expand Up @@ -4832,6 +4832,7 @@ static void end_statement_v1(PSI_statement_locker *locker, void *stmt_da)
memcpy(pfs->m_message_text, da->message(), MYSQL_ERRMSG_SIZE);
pfs->m_message_text[MYSQL_ERRMSG_SIZE]= 0;
pfs->m_sql_errno= da->sql_errno();
pfs->m_error_count++;
memcpy(pfs->m_sqlstate, da->get_sqlstate(), SQLSTATE_LENGTH);
break;
case Diagnostics_area::DA_DISABLED:
Expand Down
40 changes: 38 additions & 2 deletions storage/perfschema/pfs_timer.cc
@@ -1,4 +1,4 @@
/* Copyright (c) 2008, 2012, Oracle and/or its affiliates. All rights reserved.
/* Copyright (c) 2008, 2015, Oracle and/or its affiliates. All rights reserved.
This program is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
Expand Down Expand Up @@ -124,6 +124,42 @@ void init_timers(void)
Pick best replacements.
*/

/*
For WAIT, the cycle timer is used by default. However, it is not available
on all architectures. Fall back to the nanosecond timer in this case. It is
unlikely that neither cycle nor nanosecond are available, but we continue
probing less resolution timers anyway for consistency with other events.
*/

if (cycle_to_pico != 0)
{
/* Normal case. */
wait_timer= TIMER_NAME_CYCLE;
}
else if (nanosec_to_pico != 0)
{
/* Robustness, no known cases. */
wait_timer= TIMER_NAME_NANOSEC;
}
else if (microsec_to_pico != 0)
{
/* Robustness, no known cases. */
wait_timer= TIMER_NAME_MICROSEC;
}
else if (millisec_to_pico != 0)
{
/* Robustness, no known cases. */
wait_timer= TIMER_NAME_MILLISEC;
}
else
{
/*
Will never be reached on any architecture, but must provide a default if
no other timers are available.
*/
wait_timer= TIMER_NAME_TICK;
}

/*
For STAGE and STATEMENT, a timer with a fixed frequency is better.
The prefered timer is nanosecond, or lower resolutions.
Expand Down Expand Up @@ -174,7 +210,7 @@ void init_timers(void)
else if (millisec_to_pico != 0)
{
/* Robustness, no known cases. */
idle_timer= TIMER_NAME_MILLISEC;
wait_timer= TIMER_NAME_MILLISEC;
}
else if (tick_to_pico != 0)
{
Expand Down

0 comments on commit 2ca4141

Please sign in to comment.