The patch on postgres is selecting every row from the table before updating

[andymoon]

I believe the line

      return this._getOrFind(id, options)

should be

      return this._getOrFind(id, params)

https://github.com/feathersjs/feathers-sequelize/blob/edc431f82018b7f6f376030d05225b017f26bc41/src/index.js#L180

[daffl]

I think you're right. Can you describe the behaviour you are seeing? If it is what I think I'd be surprised that the tests (or somebody else) didn't catch it before.

[andymoon]

With the debug level logs I can see it make two queries against the table one for the count the other for every row. The queries have no where clauses while the update does. I believe the first two queries are coming from the this._getOrFind function call.

[daffl]

I think that makes sense. Could you submit a PR with the change you proposed?

[andymoon]

#169

[daffl]

Closed via #169