Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

perf: remove fetch_schema_from_transport? #339

Open
gosuto-inzasheru opened this issue Jan 27, 2025 · 0 comments
Open

perf: remove fetch_schema_from_transport? #339

gosuto-inzasheru opened this issue Jan 27, 2025 · 0 comments

Comments

@gosuto-inzasheru
Copy link
Contributor

client = Client(transport=transport, fetch_schema_from_transport=True)

@jalbrekt85 is there a reason to have fetch_schema_from_transport set to true here (and in the other instances of a client)?

what i understand from graphql-python/gql#331 is that it is actually not really needed to have it turned on in prod. maaaaybe during dev.

anyway im asking because this job halted because of an issue with the schema fetching. i think we can optimise that cron a bit by removing the fetch schema flag

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant