Skip to content

Split oversized chip ingress PublishBatch requests#2011

Closed
pkcll wants to merge 1 commit into
infoplat-3436-chipingress-publishBatchfrom
infoplat-3436-chipingress-publishBatch--split-max-grpc
Closed

Split oversized chip ingress PublishBatch requests#2011
pkcll wants to merge 1 commit into
infoplat-3436-chipingress-publishBatchfrom
infoplat-3436-chipingress-publishBatch--split-max-grpc

Conversation

@pkcll

@pkcll pkcll commented Apr 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • split chip-ingress PublishBatch sends by serialized request size
  • fail a single oversized event before calling PublishBatch
  • add regression tests for request splitting and oversized single-event handling

Tests

  • go test ./batch
  • go test ./...
  • go build ./...

@github-actions

github-actions Bot commented Apr 29, 2026

Copy link
Copy Markdown

📊 API Diff Results

No changes detected for module github.com/smartcontractkit/chainlink-common/pkg/chipingress

View full report

@github-actions

Copy link
Copy Markdown

📊 API Diff Results

No changes detected for module github.com/smartcontractkit/chainlink-common

View full report

@pkcll pkcll force-pushed the infoplat-3436-chipingress-publishBatch--split-max-grpc branch from 6972cd3 to 6032199 Compare April 29, 2026 13:41
@pkcll pkcll force-pushed the infoplat-3436-chipingress-publishBatch--split-max-grpc branch from 6032199 to 6212192 Compare April 29, 2026 13:42
@pkcll

pkcll commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

Its already in the main branch

@pkcll pkcll closed this Jun 9, 2026
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.

1 participant