Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug report] Gravitrino Web UI gives error when looking at metalake details. #1150

Closed
justinmclean opened this issue Dec 14, 2023 · 1 comment · Fixed by #1330
Closed

[Bug report] Gravitrino Web UI gives error when looking at metalake details. #1150

justinmclean opened this issue Dec 14, 2023 · 1 comment · Fixed by #1330
Assignees

Comments

@justinmclean
Copy link
Member

Describe what's wrong

An error occurs when clicking on the details tab.

Error message and/or stacktrace

Unhandled Runtime Error
TypeError: null is not an object (evaluating 'activatedItem.comment')
Call Stack
DetailsView

file:///Users/justin/gravitino/web/dist/static/chunks/baa7b3..js (1859:52)
renderWithHooks

node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (11009:2)
mountIndeterminateComponent

node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (16761:12)
beginWork

node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (26741:13)
performUnitOfWork

node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (25587:4)
workLoopSync

node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (25303:2)
renderRootSync

node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (25258:6)
recoverFromConcurrentError

node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (24476:34)
performSyncWorkOnRoot

node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (24739:4)
flushSyncWorkAcrossRoots_impl

node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (10274:26)
flushSyncWorkOnAllRoots

node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (10234:0)
processRootScheduleInMicrotask

node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (10379:0)

node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (10550:4)

How to reproduce

  1. Start up Gravitoino playground.
  2. Start Web UI with "yarn server".
  3. Click on "metalake_demo".
  4. Click on details.

A similar error occurs for other entities when clicking on the details tab.

Additional context

No response

@justinmclean justinmclean changed the title [Bug report] Gravitrino Web UI give error when look at meta lake details. [Bug report] Gravitrino Web UI gives error when looking at metalake details. Dec 14, 2023
@justinmclean
Copy link
Member Author

Note this error occurs on Safari (16.6) but doesn't occur on Chrome (Version 120.0.6099.109)

@jerryshao jerryshao added this to the Gravitino 0.4.0 milestone Dec 14, 2023
jerryshao pushed a commit that referenced this issue Jan 4, 2024
…ck (#1330)

### What changes were proposed in this pull request?

The error occurred due to the failure to locate the correct variable
object and its properties.

Resolve: use an empty string as a substitute.

Extra:
1. Initialize and reset the tree list when the metalake page is loaded.
2. Improve rendering of components check.

### Why are the changes needed?

Fix: #1150 

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

N/A
github-actions bot pushed a commit that referenced this issue Jan 4, 2024
…ck (#1330)

### What changes were proposed in this pull request?

The error occurred due to the failure to locate the correct variable
object and its properties.

Resolve: use an empty string as a substitute.

Extra:
1. Initialize and reset the tree list when the metalake page is loaded.
2. Improve rendering of components check.

### Why are the changes needed?

Fix: #1150 

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

N/A
jerryshao pushed a commit that referenced this issue Jan 4, 2024
…ck (#1334)

### What changes were proposed in this pull request?

The error occurred due to the failure to locate the correct variable
object and its properties.

Resolve: use an empty string as a substitute.

Extra:
1. Initialize and reset the tree list when the metalake page is loaded.
2. Improve rendering of components check.

### Why are the changes needed?

Fix: #1150 

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

N/A

Co-authored-by: CHEYNE <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants