search

bminer / node-mysql-queues

Wraps 'node-mysql' to provide mulitple query queues, allowing support for multiple statements and transactions.
MIT License
92 stars 11 forks source link

Is calling execute() required? #11

Closed pixelfreak closed 12 years ago

pixelfreak commented 12 years ago

In your docs, the code for transaction after "//Or... as of version 0.3.0, you can do this..." does not have execute() after commit(). Is this intentional?

bminer commented 12 years ago

Good question. Technically, yes; it was intentional. I was intentionally being lazy. :) The documentation notes that commit() implicitly calls resume(), which calls execute(). So, in this particular case, an explicit execute() call is not required.