[KYUUBI #6069] [TEST] TFrontendServiceSuite - close expired operations
# 🔍 Description
## Issue References 🔗
This pull request fixes #
## Describe Your Solution 🔧
Remove unnecessary waits
## Types of changes 🔖
- [x] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
## Test Plan 🧪
#### Behavior Without This Pull Request ⚰️
Flasy test
#### Behavior With This Pull Request 🎉
GA
#### Related Unit Tests
TFrontendServiceSuite - "close expired operations"
---
# Checklist 📝
- [ ] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)
**Be nice. Be informative.**
Closes #6069 from cxzl25/fix_TF_close_op_test.
Closes #6069
5895d098e [sychen] trigger test
a01f67c48 [sychen] trigger test
d67f2ea20 [sychen] test
Authored-by: sychen <sychen@ctrip.com>
Signed-off-by: Cheng Pan <chengpan@apache.org>
(cherry picked from commit 43b5431005)
Signed-off-by: Cheng Pan <chengpan@apache.org>
This commit is contained in:
parent
f9556b305e
commit
fcc40ee9a6
@ -557,9 +557,7 @@ class TFrontendServiceSuite extends KyuubiFunSuite {
|
||||
assert(cancelOpResp.getStatus.getStatusCode === TStatusCode.SUCCESS_STATUS)
|
||||
assert(sessionManager.getOpenSessionCount === 1)
|
||||
assert(session.lastIdleTime === 0)
|
||||
eventually(timeout(Span(60, Seconds)), interval(Span(1, Seconds))) {
|
||||
assert(lastAccessTime < session.lastAccessTime)
|
||||
}
|
||||
|
||||
lastAccessTime = session.lastAccessTime
|
||||
|
||||
eventually(timeout(Span(60, Seconds)), interval(Span(1, Seconds))) {
|
||||
|
||||
Loading…
Reference in New Issue
Block a user