feathersjs / feathers

The API and real-time application framework
https://feathersjs.com
MIT License
15.03k stars 750 forks source link

.returning() is not supported by mysql and will not have any effect. #2989

Closed 8BallBomBom closed 1 year ago

8BallBomBom commented 1 year ago

Randomly came across this warning. Seems the issue that was reported and fixed before Knex was moved over is occurring again.

daffl commented 1 year ago

Probably reintroduced via https://github.com/feathersjs/feathers/pull/2934 but for a different reason. Might have to be reverted again - it was a fix for allowing database functions that generated ids (like UUIDs) but I think the recommendation should now just be to do it in a resolver.