[apache#5650] improvement(web): create catalog page minor change (apa…

…che#5653) ### What changes were proposed in this pull request? - Capitalized display type values. - Sorted the provider list alphabetically. ### Why are these changes needed? Fix: apache#5650 ### Does this PR introduce any user-facing changes? No. ### How was this patch tested? Opened the "Create Catalog" dialog and verified the changes.
xunliu · Nov 26, 2024 · 59d0628 · 59d0628
1 parent 4ddd885
commit 59d0628
Show file tree

Hide file tree

Showing 2 changed files with 101 additions and 101 deletions.
diff --git a/web/web/src/app/metalakes/metalake/rightContent/CreateCatalogDialog.js b/web/web/src/app/metalakes/metalake/rightContent/CreateCatalogDialog.js
@@ -525,9 +525,9 @@ const CreateCatalogDialog = props => {
                       disabled={type === 'update'}
                       data-refer='catalog-type-selector'
                     >
-                      <MenuItem value={'relational'}>relational</MenuItem>
-                      <MenuItem value={'fileset'}>fileset</MenuItem>
-                      <MenuItem value={'messaging'}>messaging</MenuItem>
+                      <MenuItem value={'relational'}>Relational</MenuItem>
+                      <MenuItem value={'fileset'}>Fileset</MenuItem>
+                      <MenuItem value={'messaging'}>Messaging</MenuItem>
                     </Select>
                   )}
                 />

diff --git a/web/web/src/lib/utils/initial.js b/web/web/src/lib/utils/initial.js
@@ -48,6 +48,34 @@ export const messagingProviders = [
 ]
 
 export const providers = [
+  {
+    label: 'Apache Doris',
+    value: 'jdbc-doris',
+    defaultProps: [
+      {
+        key: 'jdbc-driver',
+        value: '',
+        required: true,
+        description: 'e.g. com.mysql.jdbc.Driver'
+      },
+      {
+        key: 'jdbc-url',
+        value: '',
+        required: true,
+        description: 'e.g. jdbc:mysql://localhost:9030'
+      },
+      {
+        key: 'jdbc-user',
+        value: '',
+        required: true
+      },
+      {
+        key: 'jdbc-password',
+        value: '',
+        required: true
+      }
+    ]
+  },
   {
     label: 'Apache Hive',
     value: 'hive',
@@ -60,6 +88,26 @@ export const providers = [
       }
     ]
   },
+  {
+    label: 'Apache Hudi',
+    value: 'lakehouse-hudi',
+    defaultProps: [
+      {
+        key: 'catalog-backend',
+        value: 'hms',
+        defaultValue: 'hms',
+        required: true,
+        select: ['hms'],
+        description: 'Apache Hudi catalog type choose properties'
+      },
+      {
+        key: 'uri',
+        value: '',
+        required: true,
+        description: 'Apache Hudi catalog uri config'
+      }
+    ]
+  },
   {
     label: 'Apache Iceberg',
     value: 'lakehouse-iceberg',
@@ -135,95 +183,6 @@ export const providers = [
       }
     ]
   },
-  {
-    label: 'MySQL',
-    value: 'jdbc-mysql',
-    defaultProps: [
-      {
-        key: 'jdbc-driver',
-        value: '',
-        required: true,
-        description: 'e.g. com.mysql.jdbc.Driver or com.mysql.cj.jdbc.Driver'
-      },
-      {
-        key: 'jdbc-url',
-        value: '',
-        required: true,
-        description: 'e.g. jdbc:mysql://localhost:3306'
-      },
-      {
-        key: 'jdbc-user',
-        value: '',
-        required: true
-      },
-      {
-        key: 'jdbc-password',
-        value: '',
-        required: true
-      }
-    ]
-  },
-  {
-    label: 'PostgreSQL',
-    value: 'jdbc-postgresql',
-    defaultProps: [
-      {
-        key: 'jdbc-driver',
-        value: '',
-        required: true,
-        description: 'e.g. org.postgresql.Driver'
-      },
-      {
-        key: 'jdbc-url',
-        value: '',
-        required: true,
-        description: 'e.g. jdbc:postgresql://localhost:5432/your_database'
-      },
-      {
-        key: 'jdbc-user',
-        value: '',
-        required: true
-      },
-      {
-        key: 'jdbc-password',
-        value: '',
-        required: true
-      },
-      {
-        key: 'jdbc-database',
-        value: '',
-        required: true
-      }
-    ]
-  },
-  {
-    label: 'Apache Doris',
-    value: 'jdbc-doris',
-    defaultProps: [
-      {
-        key: 'jdbc-driver',
-        value: '',
-        required: true,
-        description: 'e.g. com.mysql.jdbc.Driver'
-      },
-      {
-        key: 'jdbc-url',
-        value: '',
-        required: true,
-        description: 'e.g. jdbc:mysql://localhost:9030'
-      },
-      {
-        key: 'jdbc-user',
-        value: '',
-        required: true
-      },
-      {
-        key: 'jdbc-password',
-        value: '',
-        required: true
-      }
-    ]
-  },
   {
     label: 'Apache Paimon',
     value: 'lakehouse-paimon',
@@ -300,22 +259,30 @@ export const providers = [
     ]
   },
   {
-    label: 'Apache Hudi',
-    value: 'lakehouse-hudi',
+    label: 'MySQL',
+    value: 'jdbc-mysql',
     defaultProps: [
       {
-        key: 'catalog-backend',
-        value: 'hms',
-        defaultValue: 'hms',
+        key: 'jdbc-driver',
+        value: '',
         required: true,
-        select: ['hms'],
-        description: 'Apache Hudi catalog type choose properties'
+        description: 'e.g. com.mysql.jdbc.Driver or com.mysql.cj.jdbc.Driver'
       },
       {
-        key: 'uri',
+        key: 'jdbc-url',
         value: '',
         required: true,
-        description: 'Apache Hudi catalog uri config'
+        description: 'e.g. jdbc:mysql://localhost:3306'
+      },
+      {
+        key: 'jdbc-user',
+        value: '',
+        required: true
+      },
+      {
+        key: 'jdbc-password',
+        value: '',
+        required: true
       }
     ]
   },
@@ -346,6 +313,39 @@ export const providers = [
         required: true
       }
     ]
+  },
+  {
+    label: 'PostgreSQL',
+    value: 'jdbc-postgresql',
+    defaultProps: [
+      {
+        key: 'jdbc-driver',
+        value: '',
+        required: true,
+        description: 'e.g. org.postgresql.Driver'
+      },
+      {
+        key: 'jdbc-url',
+        value: '',
+        required: true,
+        description: 'e.g. jdbc:postgresql://localhost:5432/your_database'
+      },
+      {
+        key: 'jdbc-user',
+        value: '',
+        required: true
+      },
+      {
+        key: 'jdbc-password',
+        value: '',
+        required: true
+      },
+      {
+        key: 'jdbc-database',
+        value: '',
+        required: true
+      }
+    ]
   }
 ]