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

Cleanup sqlite #1139

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Cleanup sqlite #1139

wants to merge 1 commit into from

Conversation

al8n
Copy link
Collaborator

@al8n al8n commented Jun 24, 2024

No description provided.

@al8n al8n requested a review from iduartgomez June 24, 2024 08:26
@@ -28,7 +28,7 @@ impl Pool {
#[cfg_attr(feature = "redb", allow(unused))]
pub async fn new(db_dir: Option<&Path>) -> Result<Self, SqlDbError> {
let opts = if let Some(db_dir) = db_dir {
let file = db_dir.join("freenet.db");
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did you try running with sqlite to see things initialise properly?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried to use sqlx with sqlite only in standalone project, and find that this only a file. But not try it with the freenet cli.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Compile and install locally freenet to use SQLite and not Redb, install and try run ping contract to see everything works correctly before merging

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 this pull request may close these issues.

2 participants