works on GAE, in trunk. now update and delete also return the number of rows affected. The behvior o GAE is weird on gae because the number is always capped at 1000 and because the operation may take long and the process be killed in the middle.
Massimo On Dec 15, 11:56 pm, "Yarko Tymciurak" <[email protected]> wrote: > On Mon, Dec 15, 2008 at 11:07 PM, mdipierro <[email protected]> wrote: > > > I added this in trunk as suggested (in a try except to avoid > > problems). Before I make this official I would like people to run a > > test > > > db=SQLDB('.....') > > db.define_table('t',SQLField('a')) > > db.t.insert(a='test') > > print db(db.t.id>0).delete() > > > does it output 1 on > > > sqlite: YES > > postgresql: YES > > mysql: ? > > > mssql: ? > > firebird: ? > > oracle: ? > > > This is not yet implemented on GAE. If anybody knows how to, please > > send me a patch. > > > Massimo > > > On Dec 15, 6:29 pm, achipa <[email protected]> wrote: > > > Any particular reason why a shortcut to db._cursor.rowcount (or a > > > db.rowcount wrapper) is not goot enough ? I actually considered this > > > mod some time ago, but in the end I found it easier to use .rowcount, > > > as it is compatible with inserts (think 'insert into select') for > > > cases when you are branching between an update and an insert, or doing > > > an insert that results in multiple rows being inserted. Also, it's > > > potentially easier to learn/more reusable knowledge as it works the > > > same way as regular python. > > > > On Dec 16, 12:56 am, billf <[email protected]> wrote: > > > > > The Python API Spec v2.0 contains the following: > > > > > .rowcount > > > > > This read-only attribute specifies the number of rows that > > > > the last .execute*() produced (for DQL statements like > > > > 'select') or affected (for DML statements like 'update' or > > > > 'insert'). > > > > > The attribute is -1 in case no .execute*() has been > > > > performed on the cursor or the rowcount of the last > > > > operation is cannot be determined by the interface. [7] > > > > > Note: Future versions of the DB API specification could > > > > redefine the latter case to have the object return None > > > > instead of -1. > > > > ...and, under Major changes from Version 1 to version 2... > > > > The definition of the .execute() return value changed. > > > > Previously, the return value was based on the SQL statement > > > > type (which was hard to implement right) -- it is undefined > > > > now; use the more flexible .rowcount attribute > > > > instead. Modules are free to return the old style return > > > > values, but these are no longer mandated by the > > > > specification and should be considered database interface > > > > dependent. > > > > > Consequently, I believe the change to sql.py update() and delete() > > > > methods should be to change > > > > > self._db._execute(query) > > > > > to > > > > > self._db._execute(query) > > > > return self._db._cursor.rowcount > > > > > I will test this with sqlite3 and mysql - can anyone with access to > > > > other db's confirm that rowcount is supported? > > > > > On Dec 15, 11:33 pm, billf <[email protected]> wrote: > > > > > > Ok - I will try to determine if there is a "standard" way to do this. > > > > > Can you tell me whether the drivers for the current supported > > > > > databases conform to any particular specification? Python Database > > > > > API Specification v2.0 seems the appropriate one but I'd like it > > > > > confirmed. > > > > > > On Dec 15, 7:17 pm, mdipierro <[email protected]> wrote: > > > > > > > I would very much like to include this but this has to work on all > > > > > > supported databases. Can you send me a patch? It will be included. > > > > > > > On Dec 15, 2:26 am, billf <[email protected]> wrote: > > > > > > > > I would like access to the rowcount returned from update and > > delete > > > > > > > actions. There are a couple of reasons: > > > > > > > > a) it could be used to detect rare errors, e.g. expect 1 row to > > be > > > > > > > updated and zero returned. > > > > > > > > b) if the update/delete affects several rows, I'd like to know > > how > > > > > > > many rows were updated without running a separate select, e.g. > > > rowcount=db(db.employee.role==request.vars.role).update(request.vars) > > > > > > > > I believe that this can be achieved quite simply by a patch to > > sql.py > > > > > > > update() and delete() methods. > > > > > > > > Change "self._db._execute(query)" to "return > > self._db._execute(query)" > > > > > > > > If you wanted to access "rowcount" with SQLFORM then other > > changes > > > > > > > would be required. > > > > > > > > My questions are: > > > > > > > > 1) can anyone see a problem with the suggestion? > > > > > > > > 2) does anyone object to including the proposed patch? --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "web2py Web Framework" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/web2py?hl=en -~----------~----~----~----~------~----~------~--~---
