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

Display better error message when DB storage version doesn't align with Explorer's expectations #166

Closed
prrao87 opened this issue Jul 7, 2024 · 2 comments
Assignees

Comments

@prrao87
Copy link
Member

prrao87 commented Jul 7, 2024

If a user is trying to access a dev version of a Kùzu database whose storage version isn't compatible with the release build of Kùzu Explorer, the UI doesn't show a helpful error message. AN example is shown below:

[21:26:57.994] INFO (1): Access mode: READ_WRITE [21:26:58.019] ERROR (1): Error getting version of Kùzu: Error: std::bad_alloc

We should show a more helpful error message, similar to how we do in our CLI to inform the user that they should use a format of Kùzu that's compatible with the database storage version.

@mewim
Copy link
Member

mewim commented Jul 8, 2024

Explorer itself has the correct infrastructure to handle a storage version mismatch, as shown below:

Screenshot 2024-07-08 at 8 52 12 AM

However, I think the issue was that the dev builds do not actively maintain the storage version number. The storage version number is defined at: https://github.com/kuzudb/kuzu/blob/9df0a994535829df598dbfdf135795513554850c/src/include/storage/storage_version_info.h#L15-L20

For each stable release, we manually update this dictionary to add the version number of kuzu and storage. But since each dev build is automatically built and deployed, it does not get its own storage version number. Instead, it simply assumes that it has the latest storage version number in the dictionary.

@prrao87
Copy link
Member Author

prrao87 commented Jul 9, 2024

BTW, Paco faced the exact same issue yesterday, so it's clear that we have to document this on our Kùzu Explorer page so that people know that the dev build doesn't have version numbers. I'll track this as an issue there.

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

No branches or pull requests

3 participants