Skip to content

Conversation

@kumaraditya303
Copy link
Contributor

@kumaraditya303kumaraditya303 commented Nov 8, 2022

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @kumaraditya303 for commit 5435423 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-botbedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 8, 2022
Copy link
Member

@corona10corona10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I check the leak and solution through

./python.exe -m test test_finalization -R 3:3 Raised RLIMIT_NOFILE: 256 -> 1024 0:00:00 load avg: 3.73 Run tests sequentially 0:00:00 load avg: 3.73 [1/1] test_finalization beginning 6 repetitions 123456 ...... == Tests result: SUCCESS == 1 test OK. Total duration: 1.0 sec Tests result: SUCCESS 

But please take a look at #99236 (comment) nit message.

Copy link
Member

@corona10corona10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@kumaraditya303@bedevere-bot@corona10