Skip to content

Commit

Permalink
[KYUUBI #5573] Delete parts of the Kyuubi Web UI that are not useful
Browse files Browse the repository at this point in the history
### _Why are the changes needed?_

As title and make Web UI more clean.

And as Contact Us page and Overview page will do refactor later, so remain these.

### _How was this patch tested?_
- [x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

- [ ] Add screenshots for manual tests if appropriate

- [ ] [Run test](https://kyuubi.readthedocs.io/en/master/contributing/code/testing.html#running-tests) locally before make a pull request

![截屏2023-10-31 15 30 37](https://github.com/apache/kyuubi/assets/52876270/443feaf5-2d9a-4683-9214-6b7f5b5769cd)

### _Was this patch authored or co-authored using generative AI tooling?_

No

Closes #5574 from zwangsheng/KYUUBI#5573.

Closes #5573

462f9f6 [zwangsheng] fix comments
d321010 [zwangsheng] [KYUUBI #5573][Improvement] Delete parts of the Kyuubi Web UI that are not useful

Authored-by: zwangsheng <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
  • Loading branch information
zwangsheng authored and pan3793 committed Oct 31, 2023
1 parent 058d3ee commit 591250c
Show file tree
Hide file tree
Showing 11 changed files with 0 additions and 325 deletions.
47 changes: 0 additions & 47 deletions kyuubi-server/web-ui/src/layout/components/aside/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -43,58 +43,11 @@ export const MENUS = [
}
]
},
{
label: 'Workload',
icon: 'List',
children: [
{
label: 'Analysis',
icon: 'VideoPlay',
router: '/workload/analysis'
},
{
label: 'Queue',
icon: 'Select',
router: '/workload/queue'
},
{
label: 'Session',
icon: 'Select',
router: '/workload/session'
},
{
label: 'Query',
icon: 'Select',
router: '/workload/query'
}
]
},
{
label: 'Operation',
icon: 'List',
children: [
{
label: 'Running Jobs',
icon: 'VideoPlay',
router: '/operation/runningJobs'
},
{
label: 'Completed Jobs',
icon: 'Select',
router: '/operation/completedJobs'
}
]
},
{
label: 'Swagger',
icon: 'List',
router: '/swagger'
},
{
label: 'Contact Us',
icon: 'PhoneFilled',
router: '/contact'
},
{
label: 'SQL Lab',
icon: 'Cpu',
Expand Down
26 changes: 0 additions & 26 deletions kyuubi-server/web-ui/src/router/contact/index.ts

This file was deleted.

6 changes: 0 additions & 6 deletions kyuubi-server/web-ui/src/router/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,6 @@

import { createRouter, createWebHistory } from 'vue-router'
import overviewRoutes from './overview'
import workloadRoutes from './workload'
import operationRoutes from './operation'
import contactRoutes from './contact'
import managementRoutes from './management'
import detailRoutes from './detail'
import swaggerRoutes from './swagger'
Expand All @@ -40,12 +37,9 @@ const routes = [
redirect: 'overview',
children: [
...overviewRoutes,
...workloadRoutes,
...operationRoutes,
...managementRoutes,
...detailRoutes,
...swaggerRoutes,
...contactRoutes,
...labRoutes
]
}
Expand Down
31 changes: 0 additions & 31 deletions kyuubi-server/web-ui/src/router/operation/index.ts

This file was deleted.

41 changes: 0 additions & 41 deletions kyuubi-server/web-ui/src/router/workload/index.ts

This file was deleted.

29 changes: 0 additions & 29 deletions kyuubi-server/web-ui/src/views/operation/completedJobs/index.vue

This file was deleted.

29 changes: 0 additions & 29 deletions kyuubi-server/web-ui/src/views/operation/runningJobs/index.vue

This file was deleted.

29 changes: 0 additions & 29 deletions kyuubi-server/web-ui/src/views/workload/analysis/index.vue

This file was deleted.

29 changes: 0 additions & 29 deletions kyuubi-server/web-ui/src/views/workload/query/index.vue

This file was deleted.

29 changes: 0 additions & 29 deletions kyuubi-server/web-ui/src/views/workload/queue/index.vue

This file was deleted.

29 changes: 0 additions & 29 deletions kyuubi-server/web-ui/src/views/workload/session/index.vue

This file was deleted.

0 comments on commit 591250c

Please sign in to comment.