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

explore Batch methods https://google.aip.dev/233 #211

Open
glimchb opened this issue Nov 17, 2022 · 6 comments
Open

explore Batch methods https://google.aip.dev/233 #211

glimchb opened this issue Nov 17, 2022 · 6 comments
Labels
Storage APIs or code related to storage area

Comments

@glimchb
Copy link
Member

glimchb commented Nov 17, 2022

https://google.aip.dev/233

@mestery
Copy link
Contributor

mestery commented Nov 17, 2022

Did you also look at gRPC streaming? We use this in sessionOffload.

@glimchb
Copy link
Member Author

glimchb commented Nov 17, 2022

how streaming helps ? we want to create volume and expose it to the host in a single request...

@mestery
Copy link
Contributor

mestery commented Nov 17, 2022

Streaming won't help for that use case.

To be fair, this issue had literally zero context. If you had put that context in the issue, I would not have suggested it. 😉

@glimchb
Copy link
Member Author

glimchb commented Nov 17, 2022

you are 100% right ;-)

@pathreya
Copy link
Contributor

pathreya commented Nov 17, 2022

Can some of the methods to be batched be optional i.e. they may or may not succeed.

@benlwalker
Copy link

Batching is only for creating large sets of the same resource. We want to batch together multiple different resource creation steps. These are not the same thing and batching doesn't solve the problem behind the desire to add transaction support.

@glimchb glimchb added the Storage APIs or code related to storage area label Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Storage APIs or code related to storage area
Projects
None yet
Development

No branches or pull requests

4 participants