fix to avoid requery database for large result set via get_row

[openadvent]

when we have large results set,
using get_row to loop through results with offset to get the returned results.
but calling get_row seems to query the database on every call,
so im suggesting that it will detect a select statement and if it matches last query,
avoid requery database and just return the result

[openadvent]

wp-db.php

[openadvent]

updated wpdb

[nacin]

Hi openadvent, thanks for the report. In the future, if you can submit a patch formatted for SVN or Git, that would be great. There's more in the ​handbook.

This isn't something we're going to add to WordPress. If you're calling the database API, there is an expectation that the query will run and return the latest results. (Keep in mind, for example, other processes could be making DB changes).

You should optimize your code to not unnecessarily run DB statements you don't need. Rather than using get_row(), get_col(), or get_var() with offsets, use get_results() and run through the results yourself.

[openadvent]

hi,

the get_results uses too much memory for big results

unless you have another suggestion for get_row, maybe an alternative name to loop through records via offset

[nacin]

Then put a LIMIT on it. get_row() for X results and get_results() for X results are going to use the same amount of memory.

If you want to loop through records by "chunking" them into multiple queries using offset LIMITs, then you can/should do so. You don't need an API for that; it's too restricting for what needs to be handled on a case-by-case basis.