Patrick Strawderman

Results 46 comments of Patrick Strawderman

You're right. I've added support for it in #1800

Hey @ehardy, thank you for the PR and apologies for the slow movement on this. There is a larger refactor of the `InputObjectMapper` that I hope to land soon that...

I confirmed locally with the example Kotlin apps that exceptions thrown from suspend functions are no longer wrapped in `InvocationTargetException`.

Could we take a step back and discuss how this is intended to work end-to-end? What it sounds like to me in this proposal is that the `DefaultInputObjectMapper` would need...

I'd just mention that you can do this today by just checking the `DataFetchingEnvironment`. E.g., ```kotlin dfe.getArgument("input").containsKey("foo") ``` would tell you whether a null value was supplied explicitly or not....

> * DGS runtime: DefaultInputObjectMapper shouldn't be changed to accommodate this feature, especially calling a private method using reflection > * DGS CodeGen: > > * Avoid bitset, instead create...

Oh, one other issue, is that mapping for Kotlin types doesn't use setters at all currently. There is a [ticket open](https://github.com/Netflix/dgs-framework/issues/1057) for that, I think we can fix it in...

> There will be absolutely NO changes to ObjectMappers. Any JSON deserializer (not only Jackson, but any other) will work with this out of the box. All changes are encapsulated...

> > Oh, one other issue, is that mapping for Kotlin types doesn't use setters at all currently. There is a [ticket open](https://github.com/Netflix/dgs-framework/issues/1057) for that, I think we can fix...