Skip to content

Request Mirroring Proposal#540

Open
shadialtarsha wants to merge 1 commit into
grpc:masterfrom
shadialtarsha:request-mirroring
Open

Request Mirroring Proposal#540
shadialtarsha wants to merge 1 commit into
grpc:masterfrom
shadialtarsha:request-mirroring

Conversation

@shadialtarsha
Copy link
Copy Markdown

Proposal to support xds Request Mirroring in grpc.

@shadialtarsha
Copy link
Copy Markdown
Author

I would appreciate some eyes on this PR 🙏🏻

@shadialtarsha
Copy link
Copy Markdown
Author

Hey @markdroth, I would appreciate some guidance on how I should navigate the review process of this proposal 😅

@markdroth
Copy link
Copy Markdown
Member

Sorry for the delay -- I'm really swamped at the moment and haven't yet had time to look at this. I hope to take a review pass sometime next week.

@shadialtarsha
Copy link
Copy Markdown
Author

Really appreciate it @markdroth

@shadialtarsha
Copy link
Copy Markdown
Author

@markdroth just a friendly reminder 🙈

@shadialtarsha
Copy link
Copy Markdown
Author

Hey @markdroth, could you please help loop in other maintainers on this PR if you don’t have bandwidth to review it?

I’m trying to figure out the best way to unblock it, and this proposal is quite important for the work we’re planning on our side. Any help or direction would be greatly appreciated!

@shadialtarsha
Copy link
Copy Markdown
Author

Hey @markdroth, just following up here again 🙂

Would it make sense for me to start working on an implementation draft/prototype in parallel while the proposal is being reviewed?

This proposal would unblock some important work we’re planning internally, and I’d be happy to help push it forward however makes the most sense. If there are other maintainers I should loop in as well, I’d really appreciate the guidance.

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