-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
Consider introduce the awaitility framework to replace Thread.sleep in E2E #24106
Comments
Hi, @zhaojinchao95, i want to have a try, please assign to me, thanks! |
Welcome, which task do you do first? |
Hi, @zhaojinchao95, I'll start from BTW, It seems that there is no such module |
I think your idea are very good, |
@liuxiaocs7 Hi, any update? |
Hi, @zhaojinchao95 , sorry for late and thanks for your reminder, I've been a little busy recently so I'm a little behind schedule. I'm free now and I'll keep updating this week! |
Will this issue be completed before June 15 which is the final date of version 5.4.0? |
There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale. |
…RDLE2EIT class (#27536) * Refactor: test-e2e-sql use the awaitility framework replace sleep in RDLE2EIT class (#24106) * Fix code style * add SqlCommandType judgment in SingleE2EContainerComposer and E2ETestParameterGenerator class * Fix code style * pollDelay() replace atMost() in the awaitility framework * Fix code style --------- Co-authored-by: wyx <=1105017602@qq.com>
There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale. |
Hi community,
Now, many
Thread.sleep
code inE2E
IT. It's aim to waiting meta data refresh. But sometimes, we needn't wait that long time, thus it increases IT run time. So we consider introduce awaitility to optimization it.Task list :
The text was updated successfully, but these errors were encountered: