Bug #28551 | The warning 'No database selected' is reported when calling stored procedures | ||
---|---|---|---|
Submitted: | 21 May 2007 7:09 | Modified: | 22 Jul 2007 18:23 |
Reporter: | Asuka Kenji Siu Ching Pong (Basic Quality Contributor) (OCA) | Email Updates: | |
Status: | Closed | Impact on me: | |
Category: | MySQL Server: Stored Routines | Severity: | S2 (Serious) |
Version: | 5.1.18-beta, 5.0 BK | OS: | Any |
Assigned to: | Konstantin Osipov | CPU Architecture: | Any |
Tags: | backport_050044SP1, No database selected, regression, stored procedure |
[21 May 2007 7:09]
Asuka Kenji Siu Ching Pong
[21 May 2007 8:14]
Sveta Smirnova
Thank you for the report. Verified as described. May be duplicate of bug #13616
[21 May 2007 8:20]
Sveta Smirnova
Bug doesn't exists in versions 5.0.41 an 5.1.17
[4 Jul 2007 22:20]
Bugs System
A patch for this bug has been committed. After review, it may be pushed to the relevant source trees for release in the next version. You can access the patch from: http://lists.mysql.com/commits/30340 ChangeSet@1.2514, 2007-07-05 02:20:32+04:00, kostja@bodhi.(none) +4 -0 A fix and a teset case for Bug#28551 The warning 'No database selected' is reported when calling stored procedures Remove the offending warning introduced by the fix for Bug 25082 This minimal patch relies on the intrinsic knowledge of the fact that mysql_change_db is never called with 'force_switch' set to TRUE when such a warning may be needed: * every stored routine belongs to a database (unlike, e.g., a user defined function, which does not), so if we're activating the database of a stored routine, it can never be NULL. Therefore, this branch is never called for activation. * if we're restoring the 'old' current database after routine execution is complete, we should not issue a warning, since it's OK to call a routine without having previously selected the current database. TODO: 'force_switch' is an ambiguous flag, since we do not actually have to 'force' the switch in case of stored routines at all. When we activate the routine's database, we should perform all the checks as in case of 'use db', and so we already do (in this case 'force_switch' is unused). When we load a routine into cache, we should not use mysql_change_db at all, since there it's enough to call thd->reset_db(). We do it this way for triggers, but code for routines is different (wrongly). TODO: bugs are lurking in replication, since it bypasses mysql_change_db and calls thd->[re_]set_db to set the current database. The latter does not change thd->db_charset, thd->sctx->db_access and thd->variables.collation_database (and this may have nasty side effects). These todo items are to be addressed in a separate patch, if at all.
[12 Jul 2007 18:29]
Konstantin Osipov
Actually queued it into 5.0-runtime (a small patch).
[17 Jul 2007 15:29]
Bugs System
Pushed into 5.0.48
[17 Jul 2007 15:31]
Bugs System
Pushed into 5.1.21-beta
[22 Jul 2007 18:23]
Paul DuBois
Noted in 5.0.48, 5.1.21 changelogs. If a stored procedure was created and invoked prior to selecting a default database with USE, a No database selected error occurred.
[11 Sep 2007 21:25]
Keith Mendoza
This issue might be more an issue with the UPDATE statement, than the stored procedure. I attempted to execute an UPDATE statement involving 3 tables without first executing a USE database, and I got the same error warning. However, after I executed "use mysql" then reran the statement it worked.