1) (in-code) Documentation isn't clear on whether sql_query TTL arg is in seconds, minutes, hours, or days. following through to the ACM call shows it to be seconds, perhaps this should be noted in the @params

2) when recovering a cached query, the following dbal functionality is broken:

sql_fetchfield
sql_rowseek
sql_numrows ("not used in core code")

sql_fetchrow in ACM discards the retrieved cached data (array_shift), which breaks possible implementations of sql_rowseek and sql_fetchfield (and sql_numrows, which could be implemented fairly easily as a sizeof() call. the other two could be implemented by keeping a second array of index pointers)