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.

Sign up for GitHub

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

Jump to bottom

[ingester/stream]: Add a byte stream rate limit. #4191

Merged
merged 9 commits into from
Aug 20, 2021
Merged

[ingester/stream]: Add a byte stream rate limit. #4191

merged 9 commits into from
Aug 20, 2021

Commits on Aug 19, 2021

  1. Add a byte stream rate limit. 

    Signed-off-by: Callum Styan <callumstyan@gmail.com>
    @cstyan
    cstyan committed Aug 19, 2021
    Configuration menu
    Copy the full SHA
    c318c5c View commit details
    Browse the repository at this point in the history

  2. Track discarded lines due to stream rate limit + use ByteSize for the 

    config value.

Signed-off-by: Callum Styan <callumstyan@gmail.com>
    @cstyan
    cstyan committed Aug 19, 2021
    Configuration menu
    Copy the full SHA
    69430df View commit details
    Browse the repository at this point in the history

  3. Address review comments 

    Signed-off-by: Callum Styan <callumstyan@gmail.com>
    @cstyan
    cstyan committed Aug 19, 2021
    Configuration menu
    Copy the full SHA
    9d1849b View commit details
    Browse the repository at this point in the history

  4. Another round of review fixes. 

    Signed-off-by: Callum Styan <callumstyan@gmail.com>
    @cstyan
    cstyan committed Aug 19, 2021
    Configuration menu
    Copy the full SHA
    d5e64d4 View commit details
    Browse the repository at this point in the history

  5. Fix goimports issue. 

    Signed-off-by: Callum Styan <callumstyan@gmail.com>
    @cstyan
    cstyan committed Aug 19, 2021
    Configuration menu
    Copy the full SHA
    c89bba2 View commit details
    Browse the repository at this point in the history

  6. Fix duplicate flag registration + stream rate limit test. 

    Signed-off-by: Callum Styan <callumstyan@gmail.com>
    @cstyan
    cstyan committed Aug 19, 2021
    Configuration menu
    Copy the full SHA
    5fe344c View commit details
    Browse the repository at this point in the history

  7. Fix stream rate/burst limit flags yaml naming. 

    Signed-off-by: Callum Styan <callumstyan@gmail.com>
    @cstyan
    cstyan committed Aug 19, 2021
    Configuration menu
    Copy the full SHA
    a7aa7e4 View commit details
    Browse the repository at this point in the history

  8. Fix tests now that we have stream rate limit. 

    Signed-off-by: Callum Styan <callumstyan@gmail.com>
    @cstyan
    cstyan committed Aug 19, 2021
    Configuration menu
    Copy the full SHA
    4651dee View commit details
    Browse the repository at this point in the history

Commits on Aug 20, 2021

  1. fix some comments 

    Signed-off-by: Callum Styan <callumstyan@gmail.com>
    @cstyan
    cstyan committed Aug 20, 2021
    Configuration menu
    Copy the full SHA
    31edbb8 View commit details
    Browse the repository at this point in the history