Skip to content

Commit

Permalink
[KYUUBI #6069] [TEST] TFrontendServiceSuite - close expired operations
Browse files Browse the repository at this point in the history
# 🔍 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

5895d09 [sychen] trigger test
a01f67c [sychen] trigger test
d67f2ea [sychen] test

Authored-by: sychen <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
  • Loading branch information
cxzl25 authored and pan3793 committed Feb 21, 2024
1 parent fbe80ac commit 43b5431
Showing 1 changed file with 1 addition and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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))) {
Expand Down

0 comments on commit 43b5431

Please sign in to comment.