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

Reply via email to