(gravitino) branch main updated: [#7423] subtask(web): Not select provider for fileset catalog (#7462)

[jshao]

This is an automated email from the ASF dual-hosted git repository.

jshao pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new e757a271a7 [#7423] subtask(web): Not select provider for fileset 
catalog (#7462)
e757a271a7 is described below

commit e757a271a7d3f2a66bf7ecd2eda904232bbff28a
Author: Qian Xia <q...@datastrato.com>
AuthorDate: Tue Jun 24 19:53:39 2025 +0800

    [#7423] subtask(web): Not select provider for fileset catalog (#7462)
    
    ### What changes were proposed in this pull request?
    Not select provider for fileset catalog
    <img width="748" alt="image"
    
src="https://github.com/user-attachments/assets/bb957bb5-ce09-44ac-9833-07830f6f0838";
    />
    <img width="783" alt="image"
    
src="https://github.com/user-attachments/assets/bee5a082-9b83-4fd8-8898-5ff82ecbda1d";
    />
    
    
    ### Why are the changes needed?
    N/A
    
    Fix: #7423
    
    ### Does this PR introduce _any_ user-facing change?
    N/A
    
    ### How was this patch tested?
    manually
---
 .../metalake/rightContent/CreateCatalogDialog.js        | 17 ++++-------------
 1 file changed, 4 insertions(+), 13 deletions(-)

diff --git 
a/web/web/src/app/metalakes/metalake/rightContent/CreateCatalogDialog.js 
b/web/web/src/app/metalakes/metalake/rightContent/CreateCatalogDialog.js
index b44ba22bc7..377381d05a 100644
--- a/web/web/src/app/metalakes/metalake/rightContent/CreateCatalogDialog.js
+++ b/web/web/src/app/metalakes/metalake/rightContent/CreateCatalogDialog.js
@@ -50,7 +50,7 @@ import { yupResolver } from '@hookform/resolvers/yup'
 
 import { groupBy } from 'lodash-es'
 import { genUpdates } from '@/lib/utils'
-import { providers, filesetProviders, messagingProviders } from 
'@/lib/utils/initial'
+import { providers, messagingProviders } from '@/lib/utils/initial'
 import { nameRegex, nameRegexDesc, keyRegex } from '@/lib/utils/regex'
 import { useSearchParams } from 'next/navigation'
 
@@ -69,8 +69,6 @@ const schema = yup.object().shape({
     switch (type) {
       case 'relational':
         return schema.oneOf(providers.map(i => i.value)).required()
-      case 'fileset':
-        return schema.oneOf(filesetProviders.map(i => i.value)).required()
       case 'messaging':
         return schema.oneOf(messagingProviders.map(i => i.value)).required()
       default:
@@ -349,16 +347,12 @@ const CreateCatalogDialog = props => {
         setValue('provider', 'hive')
         break
       }
-      case 'fileset': {
-        setProviderTypes(filesetProviders)
-        setValue('provider', 'hadoop')
-        break
-      }
       case 'messaging': {
         setProviderTypes(messagingProviders)
         setValue('provider', 'kafka')
         break
       }
+      case 'fileset':
       case 'model': {
         setProviderTypes([])
         setValue('provider', '')
@@ -404,14 +398,11 @@ const CreateCatalogDialog = props => {
           providersItems = providers
           break
         }
-        case 'fileset': {
-          providersItems = filesetProviders
-          break
-        }
         case 'messaging': {
           providersItems = messagingProviders
           break
         }
+        case 'fileset':
         case 'model': {
           providersItems = []
           break
@@ -532,7 +523,7 @@ const CreateCatalogDialog = props => {
               </FormControl>
             </Grid>
 
-            {typeSelect !== 'model' && (
+            {!['model', 'fileset'].includes(typeSelect) && (
               <Grid item xs={12}>
                 <FormControl fullWidth>
                   <InputLabel id='select-catalog-provider' 
error={Boolean(errors.provider)}>