Yann Simon

Results 224 comments of Yann Simon

Although the documentation https://www.apollographql.com/docs/graphos/reference/federation/subgraph-spec#subgraph-schema-additions states to include those directives

``` [error] /home/runner/work/sangria/sangria/modules/core/src/main/scala/sangria/execution/ResultResolver.scala:157:34: missing argument list for method createLocation in class ResultResolver [error] Unapplied methods are only converted to functions when a function type is expected. [error] You can make...

scala 3 does not like that there are 2 possible implicits. ``` [error] -- [E172] Type Error: /Users/yannsimon/projects/sangria-graphql/sangria/modules/core/src/test/scala/sangria/execution/ScalarAliasSpec.scala:70:93 [error] 70 | Argument("cid", OptionInputType(ListInputType(ListInputType(OptionInputType(UUIDType))))) [error] | ^ [error] |Ambiguous given instances:...

The fix for that is quite easy. We need to tell jemalloc to use 64-KiB pages so that the application can run on all architectures: amd64, arm64 with different page...

> It used to be this way, we moved it together because local development was just super annoying with two workspaces in one repo. I forgot to update examples all...

> Yes, this would lead to CI failures when everything seemed fine locally. AFAIC, I'd be ready to pay this "annoyance price".

@mladedav do you have time to make this PR merge-able?

I'm not sure about this. For people who are aware of the downsides of buffering the body, this is a nice addition. 💯 I'm just afraid of people adopting this...

I've never used the `Query` myself, so take my input with this in consideration. The new query is looking good, and I don't see any reasons not to use it....

Thanks, I can now reproduce the issue. The solution I had by using low priority implicits sadly does not work. I don't have any solution for this.