This is an automated email from the ASF dual-hosted git repository.
chengpan pushed a commit to branch branch-1.9
in repository https://gitbox.apache.org/repos/asf/kyuubi.git
The following commit(s) were added to refs/heads/branch-1.9 by this push:
new 266b81a33 [KYUUBI #6428] kyuubi-server/web-ui should not to set a
timeout for axios requests
266b81a33 is described below
commit 266b81a33440475daed5c5b49f0c873cd216214b
Author: zRain <[email protected]>
AuthorDate: Thu May 30 12:07:53 2024 +0800
[KYUUBI #6428] kyuubi-server/web-ui should not to set a timeout for axios
requests
# :mag: Description
## Issue References ๐
This pull request fixes #6428
## Describe Your Solution ๐ง
Remove axios request timeout limitation.
## Types of changes :bookmark:
- [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 :coffin:
Ref to #6428 description.
#### Behavior With This Pull Request :tada:
Get result after a period of time.
#### Related Unit Tests
kyuubi-server/web-ui: `pnpm test` all passed.
---
# 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 #6429 from zRains/remove-axios-timeout.
Closes #6428
e947deabe [zRain] fix: remove timeout
Authored-by: zRain <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
---
kyuubi-server/web-ui/src/utils/request.ts | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/kyuubi-server/web-ui/src/utils/request.ts
b/kyuubi-server/web-ui/src/utils/request.ts
index 3b5b9c34b..5dd9cae35 100644
--- a/kyuubi-server/web-ui/src/utils/request.ts
+++ b/kyuubi-server/web-ui/src/utils/request.ts
@@ -20,9 +20,8 @@ import { useAuthStore } from '@/pinia/auth/auth'
// create an axios instance
const service = axios.create({
- baseURL: '/', // url = base url + request url
+ baseURL: '/' // url = base url + request url
// withCredentials: true, // send cookies when cross-domain requests
- timeout: 60000 // request timeout
})
// request interceptor