Skip to content

Disable gql inner retries#181

Merged
cyc60 merged 3 commits intomasterfrom
graph-timeout
Feb 6, 2026
Merged

Disable gql inner retries#181
cyc60 merged 3 commits intomasterfrom
graph-timeout

Conversation

@cyc60
Copy link
Contributor

@cyc60 cyc60 commented Feb 6, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 6, 2026 13:14
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR disables the inner retry mechanism in the GraphQL client by setting reconnecting=False and adds an execute_timeout parameter to prevent indefinite retries at the client level.

Changes:

  • Disabled automatic reconnection in the GraphQL client session
  • Added execute timeout configuration to the GQL client initialization
  • Bumped package version from v0.12.8 to v0.12.9

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
sw_utils/graph/client.py Configured GQL client with execute_timeout and disabled reconnecting to prevent inner retries
pyproject.toml Version bump to v0.12.9

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings February 6, 2026 13:47
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cyc60 cyc60 merged commit b49887e into master Feb 6, 2026
3 checks passed
@cyc60 cyc60 deleted the graph-timeout branch February 6, 2026 14:08
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

Successfully merging this pull request may close these issues.

2 participants