Daniel Harvey

Results 17 comments of Daniel Harvey
trafficstars

This behaviour comes from the `ODBC` library. `NVARCHAR` is 2 bytes per character (so supports Unicode), but `VARCHAR` is one 1 byte. Currently the library encodes all text values as...

A fix has now been merged that will use a `VARCHAR` or `CHAR` literal instead of `NVARCHAR` / `NCHAR` where the input is all ASCII characters. If the input contains...

Have added all the tests. So far they pass if: a) We use our HGE hotfix that stops us sending prepared arguments (as `cockroach` is still strict about being sent...

The prepared args for metadata are fixed and in `main` but the fix for regular queries is "just don't send the user metadata" which can't be merged because it unblocks...

All tests have been added to the suite, some fail but have been commented out. These are the tickets to track those issues: https://github.com/hasura/graphql-engine-mono/issues/5987 https://github.com/hasura/graphql-engine-mono/issues/5985 https://github.com/cockroachdb/cockroach/issues/88355

It seemed like there was an opportunity for a fix in the `schema-parsers` library here: https://github.com/hasura/graphql-engine-mono/blob/main/server/lib/schema-parsers/src/Hasura/GraphQL/Parser/Internal/Parser.hs#L189. However on closer inspection it seems more complicated than that. Currently we collect together...

Going to reduce the scope of this to begin with - it occurred because of the addition of the `_updates` fields recently. We can solve the concrete problem with a...

We'll also need to fix this in the console so users are able to do the renaming: https://github.com/hasura/graphql-engine-mono/issues/5887

Hello! This all seems fine - the `Foreign` stuff is great, and adding `spago` / `nix` is good, just don't delete the `bower.json` yet as CI uses it and I...

We have a kind checker now, this is so close.