Igor Savin
Igor Savin
@thekuom It's less straightforward than I though, due to rather cluttered logic around this place, and a peculiar way raw insert is being done in original case. If you have...
@thekuom Inserting undefined values is not allowed. If intent is to insert null, it should be passed as null.
@thekuom And what I meant in my original comment is that there is a weird mix between raw and non-raw querying in that example. Second parameter for raw query is...
Yes, problem is with knex.raw specifically. @leventov Can you explain what knex.raw is for there?
What type-provider are you using? `fastify-type-provider-zod` has extra logic in it to handle this scenario.
Considering that Oracle is generally very difficult to support in opensource ecosystem due to really baffling decisions on the Oracle side (still no easy to consume Docker image for CI...
If feature can be extended to support pgsql as well, as you've suggested, then it is significantly more attractive, though, as PG is the goto RDBMS in Node.
If it is built pgsql-first, I can see its place in knex, as long as it can be built in a not particularly invasive way (limited impact on other code)
@OlivierCavadenti Then maybe it could be a separate project under knex umbrella? Or you see benefits in convenience of development if embedded?
How do you suggest fixing it? I'm not sure one can require node_module in any other way.