Correctly Utilize Cache in Tables Flyout (#1662)

* correctly check for cache on table flyout

Signed-off-by: Sean Li <[email protected]>

* skip start loading if fail

Signed-off-by: Sean Li <[email protected]>

---------

Signed-off-by: Sean Li <[email protected]>
(cherry picked from commit 3be2d39)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

Author: github-actions[bot]

public/components/datasources/components/manage/associated_objects/associated_objects_details_flyout.tsx

@@ -226,7 +226,21 @@ export const AssociatedObjectsDetailsFlyout = ({
 
   useEffect(() => {
     if (tableDetail && !tableDetail.columns) {
-      startLoading(datasourceName, tableDetail.database, tableDetail.name);
+      try {
+        const tables = CatalogCacheManager.getTable(
+          datasourceName,
+          tableDetail.database,
+          tableDetail.name
+        );
+        if (tables?.columns) {
+          setTableColumns(tables?.columns);
+        } else {
+          startLoading(datasourceName, tableDetail.database, tableDetail.name);
+        }
+      } catch (error) {
+        console.error(error);
+        setToast('Your cache is outdated, refresh databases and tables', 'warning');
+      }
     } else if (tableDetail && tableDetail.columns) {
       setTableColumns(tableDetail.columns);
     }