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

limit a request' size #1817

Closed
whitewum opened this issue Feb 25, 2020 · 2 comments
Closed

limit a request' size #1817

whitewum opened this issue Feb 25, 2020 · 2 comments
Labels
type/feature req Type: feature request

Comments

@whitewum
Copy link
Contributor

It's possible that (by mistake) the client sends a large request (especially when the source data is dirty, and a single request is of MB or even GB size)

I don't think it's reasonable that the client or graphd should send this large request to storaged.

(Maybe a threshold to test the request size)

WDYT?

@CPWstatic
Copy link
Contributor

What do you mean Requests' size? QPS or data? The size of data can't be estimated now.

@whitewum
Copy link
Contributor Author

What do you mean Requests' size? QPS or data? The size of data can't be estimated now.

data size.

you can't estimate the read result size. But the client can reject a large insert string request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/feature req Type: feature request
Projects
None yet
Development

No branches or pull requests

2 participants