weatherman

Howdy @kzlsakal! Thanks for reporting this issue. It seems like this is due to rover not including deprecated fields when doing introspection. Tools like Apollo Explorer, etc correctly show the...

> I presume we will have to migrate them to use rover once this is merged in order to benefit from this fix. Is that correct? Correct. `apollo` still gets...

Agreed there'd be an issue introspecting something that doesn't support `includeDeprecated` for inputs. Using 2-phase introspection makes sense.

I got my hands on one of these servers that don't support `includeDeprecated` on `__Input`, so I'll make sure rover still supports that.

Even though I got my hands on a subgraph server to replicate the issue, I don't quite know how to make it work. Part of it is making a decent...

Here's the diff of this code vs `@apollo/[email protected]`. It seems like mostly types, other than a few JS changes (which were changed in the typescript code, so they make sense)....

howdy @HishamAli81! Thanks for creating this issue. Would you mind expanding on what you'd use this for? To add some detail, that particular use of `executor` is a bit of...

Thanks again @HishamAli81 for raising this issue. It's brought up a lot of good discussion internally about how to improve the gateway. At this time, we don't plan to add...

Thanks for the offer of lending a hand @HishamAli81! We aren't likely to spend any time explicitly on `executeQueryPlan` in the next few months at least. There may be some...

Thanks for the change @mmanciop ! We're having a similar debate about when to release a similar change, since it _may_ be breaking https://github.com/apollographql/federation/issues/1938. Lemme ask around a bit and...