Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update fields structure in protobuf file for RetryOptions and ConfigSource #8027

Closed
wants to merge 7 commits into from

Conversation

surgupta-msft
Copy link
Contributor

@surgupta-msft surgupta-msft commented Jan 5, 2022

Issue describing the changes in this PR

resolves #7849

The changes in this PR are done to make proto file in host consistent with an Open PR in Protobuf repo to add fields like RetryOptions and ConfigSource. If this PR looks good, then Protobuf PR will also be updated with similar structure of fields to make them consistent.

@surgupta-msft surgupta-msft marked this pull request as ready for review January 5, 2022 20:26
@surgupta-msft surgupta-msft changed the title update proto msgs Update fields structure in protobuf file for RetryOptions and ConfigSource Jan 5, 2022
@fabiocav fabiocav requested a review from kashimiz January 5, 2022 21:12
@fabiocav
Copy link
Member

fabiocav commented Jan 5, 2022

Closing this as we need more clarity on the requirements and what the changes mean.

@fabiocav fabiocav closed this Jan 5, 2022
@surgupta-msft surgupta-msft deleted the surgupta/update-proto-msgs branch October 31, 2023 16:28
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.

Worker Indexing - Protobuf changes to update RpcFunctionMetadata with more properties
2 participants