Skip to content

Commit

Permalink
[apache#6166] web(ui): load tree load data after refreshing the versi…
Browse files Browse the repository at this point in the history
…on details page (apache#6167)

### What changes were proposed in this pull request?
Reload tree data after refreshing the version details page
<img width="1352" alt="image"
src="https://github.com/user-attachments/assets/80bc7370-0052-4be8-9b6d-efbe6d97e945"
/>

### Why are the changes needed?
N/A

Fix: apache#6166

### Does this PR introduce _any_ user-facing change?
N/A

### How was this patch tested?
manually
  • Loading branch information
LauraXia123 authored Jan 9, 2025
1 parent e9d8ee7 commit c2c5565
Showing 1 changed file with 26 additions and 13 deletions.
39 changes: 26 additions & 13 deletions web/web/src/app/metalakes/metalake/MetalakeView.js
Original file line number Diff line number Diff line change
Expand Up @@ -112,26 +112,39 @@ const MetalakeView = () => {
dispatch(getSchemaDetails({ metalake, catalog, schema }))
}

if (paramsSize === 5 && catalog && schema) {
if (paramsSize === 5 && catalog && type && schema && (table || fileset || topic || model)) {
if (!store.catalogs.length) {
await dispatch(fetchCatalogs({ metalake }))
await dispatch(fetchSchemas({ metalake, catalog, type }))
}
if (table) {
dispatch(getTableDetails({ init: true, metalake, catalog, schema, table }))
}
if (fileset) {
dispatch(getFilesetDetails({ init: true, metalake, catalog, schema, fileset }))
}
if (topic) {
dispatch(getTopicDetails({ init: true, metalake, catalog, schema, topic }))
}
if (model) {
dispatch(fetchModelVersions({ init: true, metalake, catalog, schema, model }))
dispatch(getModelDetails({ init: true, metalake, catalog, schema, model }))
switch (type) {
case 'relational':
await dispatch(fetchTables({ init: true, page: 'schemas', metalake, catalog, schema }))
await dispatch(getTableDetails({ init: true, metalake, catalog, schema, table }))
break
case 'fileset':
await dispatch(fetchFilesets({ init: true, page: 'schemas', metalake, catalog, schema }))
await dispatch(getFilesetDetails({ init: true, metalake, catalog, schema, fileset }))
break
case 'messaging':
await dispatch(fetchTopics({ init: true, page: 'schemas', metalake, catalog, schema }))
await dispatch(getTopicDetails({ init: true, metalake, catalog, schema, topic }))
break
case 'model':
await dispatch(fetchModels({ init: true, page: 'schemas', metalake, catalog, schema }))
await dispatch(fetchModelVersions({ init: true, metalake, catalog, schema, model }))
await dispatch(getModelDetails({ init: true, metalake, catalog, schema, model }))
break
default:
break
}
}
if (paramsSize === 6 && version) {
if (!store.catalogs.length) {
await dispatch(fetchCatalogs({ metalake }))
await dispatch(fetchSchemas({ metalake, catalog, type }))
await dispatch(fetchModels({ init: true, page: 'schemas', metalake, catalog, schema }))
}
dispatch(getVersionDetails({ init: true, metalake, catalog, schema, model, version }))
}
}
Expand Down

0 comments on commit c2c5565

Please sign in to comment.