Jesse Rosenberger

Results 159 comments of Jesse Rosenberger

I've opened up https://github.com/apollographql/router/pull/1993 to figure out what happened between this PR and https://github.com/apollographql/router/commit/783206a93718c65d0a25ca7c930f0f64ec59df70, when tests were passing on this PR.

the last bullet is key. thanks for opening this!

Similar reproduction just using `supergraph-demo-fed2`: ### Query ```graphql query DeferVariation { __typename allProducts { delivery { ...MyFragment @defer } sku } } fragment MyFragment on DeliveryEstimates { estimatedDelivery fastestDelivery }...

Thanks, @pcmanus! The rational and historical aspect of it makes enough sense to me that I think we can get a fix in for this on our end.

@pcmanus Just want to say I very much appreciate all of your monologue and careful thought here. 😸 It sounds like we could find some smaller steps forward which incrementally...

I've renamed the title of this issue to include a more specific name, given the introduction of other Studio functionality and to reduce the potential ambiguity.

This issue wasn't meant to be closed — the work isn't done.

Worth noting that we also have https://github.com/apollographql/router/issues/1359 which touches on tracing aspects of this.

I'm going to close this as I believe it's fixed. Happy to re-open if that turns out not to be the case. Thanks for reporting this originally!

I've renamed this issue to reflect the state of the work-around, but we should use the original reproduction above to verify any eventual fix.