Skip to content

Comments

[poc] simple cluster transport protocol#32171

Closed
teskje wants to merge 2 commits intoMaterializeInc:mainfrom
teskje:cluster-transport-protocol
Closed

[poc] simple cluster transport protocol#32171
teskje wants to merge 2 commits intoMaterializeInc:mainfrom
teskje:cluster-transport-protocol

Conversation

@teskje
Copy link
Contributor

@teskje teskje commented Apr 11, 2025

Missing:

  • heartbeating
  • server fqdn checking
  • metrics
  • logging
  • cancelation

Motivation

Tips for reviewer

Checklist

  • This PR has adequate test coverage / QA involvement has been duly considered. (trigger-ci for additional test/nightly runs)
  • This PR has an associated up-to-date design doc, is a design doc (template), or is sufficiently small to not require a design.
  • If this PR evolves an existing $T ⇔ Proto$T mapping (possibly in a backwards-incompatible way), then it is tagged with a T-proto label.
  • If this PR will require changes to cloud orchestration or tests, there is a companion cloud PR to account for those changes that is tagged with the release-blocker label (example).
  • If this PR includes major user-facing behavior changes, I have pinged the relevant PM to schedule a changelog post.

@teskje teskje force-pushed the cluster-transport-protocol branch from e5564f2 to 40fef0d Compare April 11, 2025 14:27
@teskje teskje force-pushed the cluster-transport-protocol branch 3 times, most recently from dcae233 to dcb07d0 Compare April 12, 2025 20:04
@teskje teskje force-pushed the cluster-transport-protocol branch from dcb07d0 to 5acdef6 Compare April 14, 2025 08:00
@teskje teskje closed this Jul 23, 2025
@teskje teskje deleted the cluster-transport-protocol branch July 23, 2025 12:26
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