Skip to content

binder: Pre-factor out the guts of the BinderClientTransport handshake. - #12292

Closed
jdcormie wants to merge 32 commits into
grpc:masterfrom
jdcormie:isolated-process-2
Closed

jdcormie wants to merge 32 commits into
grpc:masterfrom
jdcormie:isolated-process-2

Conversation

@jdcormie

@jdcormie jdcormie commented Aug 20, 2025 •

Copy link
Copy Markdown
Member

Part 1 of fixing #12293. No behavior change intended this is a pure refactor.

jdcormie and others added 30 commits August 19, 2025 17:21
# Conflicts:
#	binder/src/main/java/io/grpc/binder/internal/BinderTransport.java
# Conflicts:
#	binder/src/main/java/io/grpc/binder/internal/BinderTransport.java
# Conflicts:
#	binder/src/main/java/io/grpc/binder/internal/BinderTransport.java
# Conflicts:
#	binder/src/main/java/io/grpc/binder/internal/BinderTransport.java
# Conflicts:
#	binder/src/main/java/io/grpc/binder/internal/BinderTransport.java
# Conflicts:
#	binder/src/main/java/io/grpc/binder/internal/BinderTransport.java
# Conflicts:
#	binder/src/main/java/io/grpc/binder/internal/BinderTransport.java
# Conflicts:
#	binder/src/main/java/io/grpc/binder/internal/BinderTransport.java
@jdcormie
jdcormie marked this pull request as ready for review September 16, 2025 21:25
@jdcormie

Copy link
Copy Markdown
Member Author

@mateusazis PTAL

Comment thread binder/src/main/java/io/grpc/binder/internal/BinderClientTransport.java Outdated
Comment thread binder/src/main/java/io/grpc/binder/internal/BinderClientTransport.java Outdated
Comment thread binder/src/main/java/io/grpc/binder/internal/BinderClientTransport.java Outdated
Comment thread binder/src/main/java/io/grpc/binder/internal/BinderClientTransport.java Outdated
Comment thread binder/src/main/java/io/grpc/binder/internal/BinderClientTransport.java Outdated
Make ClientHandshake an interface
Make the impl private/final
@jdcormie jdcormie closed this Nov 4, 2025
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Feb 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants