Skip to content

feat: subscriptions overhaul#1374

Draft
endigma wants to merge 93 commits intomasterfrom
jesse/eng-8566-new-subscription-client
Draft

feat: subscriptions overhaul#1374
endigma wants to merge 93 commits intomasterfrom
jesse/eng-8566-new-subscription-client

Conversation

@endigma
Copy link
Member

@endigma endigma commented Feb 2, 2026

  • reset to top of memory PR
  • wip: configurable clients
  • wip: wiring
  • wip: initial graphql-ws and wiring
  • renaming and refactoring
  • wip
  • wip: refactor errors, use context for shutdown
  • wip
  • restore context setup
  • wip

@coderabbitai summary

Checklist

  • I have discussed my proposed changes in an issue and have received approval to proceed.
  • I have followed the coding standards of the project.
  • Tests or benchmarks have been added or updated.

# Conflicts:
#	go.work.sum
#	v2/go.mod
#	v2/go.sum
#	v2/pkg/engine/datasource/graphql_datasource/graphql_datasource_test.go
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 2, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch jesse/eng-8566-new-subscription-client

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Reviews will stop working after February 8, 2026 if the new IP is not added to your allowlist.


Comment @coderabbitai help to get the list of available commands and usage tips.

Base automatically changed from feat/improve-memory-usage-with-arenas to master February 6, 2026 12:13
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