Patrick Strawderman

Results 46 comments of Patrick Strawderman

Sorry for the delay with this. I've gone with a different approach here that does not use a loop or switch statement at all. One caveat is that I decided...

> Hi @kilink , > i have encapsulated the logic into a builder class. > Would this commit solve your problem? > > [eb08f5f](https://github.com/spring-cloud/spring-cloud-aws/commit/eb08f5fdc870c18ea2922c8f1b83a6478a4e73a9) Something like this could work, although...

Here's an example. We have a tool that generates Java / Kotlin classes from GraphQL schemas, so given an example trivial schema: ```graphql interface Animal { name: String } type...

> Assuming this is a reasonable improvement, given the above, do you have a PR to show your thinking on this? Also (going back to the use case), not clear...

When I run your JMeter GraphQL test plan, I get ~1650 requests / sec with DGS v4.9.25. I also tried it with the latest release candidate (v4.10.0-rc.4), as we made...

Hi @AbbadoLambily, the changes have been released in [v4.10.0](https://github.com/Netflix/dgs-framework/releases/tag/v4.10.0)

I think for your custom `PageInfo`, you should just add it yourself to your schema. The pagination support will only add a `PageInfo` type if one doesn't already exist. For...

I agree that the JDK ought to handle this check itself, although it wouldn't help Guava at the moment due to `Ordering.natural()`. I'm assuming the `null` means natural ordering contract...

This looks like it was an issue between reactor-netty and reactor-core; reactor-netty likely needed 3.5.+, while on your classpath you only had reactor-core 3.4.x. You'd either want to override the...

Support for flow was added in c48492a.