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

[fix][test]Fix resource not close after method #20715

Merged
merged 1 commit into from
Jul 5, 2023
Merged

[fix][test]Fix resource not close after method #20715

merged 1 commit into from
Jul 5, 2023

Conversation

315157973
Copy link
Contributor

@315157973 315157973 commented Jul 4, 2023 •
edited
Loading

Motivation

The resources were not released after the test, resulting in "Refresh permit" keep printing on the console

Modifications

release the resource after method

Verifying this change

  • Make sure that the change passes the CI checks.

Documentation

  • doc-not-needed

Matching PR in forked repository

315157973#11

@315157973 315157973 self-assigned this Jul 4, 2023
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Jul 4, 2023
@315157973 315157973 mentioned this pull request Jul 4, 2023
Merged
@Technoboy- Technoboy- added this to the 3.1.0 milestone Jul 4, 2023
@Technoboy- Technoboy- merged commit 6101517 into apache:master Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@coderzc coderzc coderzc approved these changes

@hezhangjian hezhangjian hezhangjian approved these changes

@Technoboy- Technoboy- Technoboy- approved these changes

Assignees

@315157973 315157973

Labels
area/test doc-not-needed Your PR changes do not impact docs ready-to-test
Projects
None yet
Milestone
3.1.0
Development

Successfully merging this pull request may close these issues.
4 participants
@315157973 @Technoboy- @hezhangjian @coderzc