Skip to content

Fix flaky test related to publish buffer throttle. #6730

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
Closed
codelipenghui wants to merge 5 commits into from
Closed

Fix flaky test related to publish buffer throttle. #6730

codelipenghui wants to merge 5 commits into from

Conversation

@codelipenghui
Copy link
Contributor

@codelipenghui codelipenghui commented Apr 13, 2020

Fix flaky test related to publish buffer throttle.

Motivation

Fix flaky test related to publish buffer throttle.

Modifications

Update tests.

Verifying this change

Test fixes

Does this pull request potentially affect one of the following parts:

If yes was chosen, please highlight the changes

  • Dependencies (does it add or upgrade a dependency): (no)
  • The public API: (no)
  • The schema: (no)
  • The default values of configurations: (no)
  • The wire protocol: (no)
  • The rest endpoints: (no)
  • The admin cli options: (no)
  • Anything that affects deployment: (no)

Documentation

  • Does this pull request introduce a new feature? (no)

@codelipenghui codelipenghui self-assigned this Apr 13, 2020
@codelipenghui codelipenghui requested review from jiazhai and sijie April 13, 2020 10:32
@codelipenghui codelipenghui added this to the 2.6.0 milestone Apr 13, 2020
@jiazhai
Copy link
Member

jiazhai commented Apr 16, 2020

/pulsarbot run-failure-checks

1 similar comment
@sijie
Copy link
Member

sijie commented Apr 21, 2020

/pulsarbot run-failure-checks

@codelipenghui codelipenghui force-pushed the fix_publish_buffer_flaky_test branch from 410daf9 to 398d51f Compare May 9, 2020 14:48
@codelipenghui codelipenghui removed this from the 2.6.0 milestone Jun 2, 2020
@codelipenghui codelipenghui deleted the fix_publish_buffer_flaky_test branch May 19, 2021 05:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sijie sijie sijie approved these changes

@jiazhai jiazhai jiazhai approved these changes

Assignees

@codelipenghui codelipenghui

Labels

area/test type/flaky-tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@codelipenghui @jiazhai @sijie