getValidator shouldn't try to second-guess a cache key

[kapouer]

Following what ajv now does, objection should trust the Map:

objection.js/lib/model/AjvValidator.js

Line 91 in 8e8ff2f

const createCacheKey = this.ajvOptions.serialize || JSON.stringify;

It should be something like

let validators = this.cache.get(jsonSchema)

Indeed ajvOptions.serialize now triggers a warning.

Should the team accept it, I'd submit a PR.
This issue won't autodestruct.

[kapouer]

AFAICT the performance is better, since no one will be able to fall for the JSON.stringify trap.
Comparing Map.get(jsonSchema) <-> Map.get(cacheKey) gives the same results.