Hi!
I've tried with latest mysql library and as I suspected, most of the
issues still stay.
The ones I'm most worried about are:
1. mysqli_stmt_num_rows() is expected to return number of rows after all
rows were fetched while returning 0 before that. libmysql definitely
doesn't do that, in full accordance with mysql docs which state row
numbers can't be had for unbuffered queries.
If we do want this new non-portable semantics in mysqlnd, we should have
it in separate test, clearly marked as such, even though I'm not
convinced having such semantics is a good idea at all (would lead to
applications working with mysqlnd and not with libmysql - invitation for
trouble).
2. last_insert_id semantics change. The test expects select queries to
reset last insert it to 0, which does not happen with libmysql. Maybe it
happens on later versions of mysql server, I do not know - but we can't
have a test the is bound to specific server version, at least not
without a check, and even better - separating it from portable tests. If
it's specific to mysqlnd it's even worse - it means that some code that
assumes libmysql behavior will be broken, and as last_insert_id() is a
very frequently used function I think it's very dangerous.
Also, new issue added with recent libmysql - newer builds of libmysql do
not declare DBUG_OFF at all in my_config.h. See for example
http://bugs.mysql.com/bug.php?id=60872. So we should take the absence of
DBUG_OFF as a sign that debug is enabled. I would recommend checking if
DEBUG_ON is defined and enabling debug only then.
--
Stanislav Malyshev, Software Architect
SugarCRM: http://www.sugarcrm.com/
(408)454-6900 ext. 227
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
