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

WIP: Perf/auto batch flush #2938

Closed
wants to merge 16 commits into from

Conversation

leisurelyrcxf
Copy link

@leisurelyrcxf leisurelyrcxf commented Aug 2, 2024

Refer to #2945

Make sure that:

  • [Y] You have read the contribution guidelines.
  • [Y] You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
  • [Y] You applied code formatting rules using the mvn formatter:format target. Don’t submit any formatting related changes.
  • [Y] You submit test cases (unit or integration tests) that back your changes.

@leisurelyrcxf leisurelyrcxf reopened this Aug 6, 2024
@leisurelyrcxf leisurelyrcxf changed the title Perf/auto batch flush WIP: Perf/auto batch flush Aug 6, 2024
@leisurelyrcxf
Copy link
Author

to add UT

@leisurelyrcxf leisurelyrcxf force-pushed the perf/auto-batch-flush branch from 2da6443 to ed60773 Compare August 7, 2024 13:10
@tishun
Copy link
Collaborator

tishun commented Aug 8, 2024

The failing tests are failing in other PRs too, seems to be a regression, investigating currently ...

@tishun
Copy link
Collaborator

tishun commented Aug 8, 2024

The failing tests are failing in other PRs too, seems to be a regression, investigating currently ...

You need to fetch the fix from #2949

@okg-cxf
Copy link
Contributor

okg-cxf commented Aug 8, 2024

Recreated with a different account, see #2950

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.

3 participants