Icebob

Good idea, a PR would be good, thanks!

Because the default id field name is `id` in sequelize adapter.

It's a breaking change, because the search result will be different after this PR. Would be better to use a new param to control the case sensitivity.

Please create a fix PR and I will help with tests

Btw, maybe this PR will conflict with https://github.com/moleculerjs/moleculer-db/pull/370 ping @thib3113

As I see the mongoose connection handling is not an easy problem. All adapters (except mongoose) create a custom connection to the db. So it would be good to follow...

I don't want to kill any common use-case. My suggestion is to add a `reuseModelConnection: true|false` to the service settings (similar to [useNativeMongooseVirtuals](https://github.com/moleculerjs/moleculer-db/blob/master/packages/moleculer-db-adapter-mongoose/src/index.js#L42)) and handle the connection by this setting....

1. Here is the calling of `entityChanged`: https://github.com/moleculerjs/moleculer-db/blob/0b5a04a8818dbbc6bfd9ebf0c113b6854d26b442/packages/moleculer-db/src/index.js#L1032 2. You don't need to call `exec` because the returned object is a `Promise`, so if you call it with `await` it...

The current version supports v6, we can support v7, but in this case we should release it in a new major version.

Ok, plz create a PR and we can discuss your suggestion with other TS guys.