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

wip: change interactive_one to read/write from/to FS device. #330

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

samwillis
Copy link
Collaborator

No description provided.

Copy link
Contributor

github-actions bot commented Sep 17, 2024

Base automatically changed from pmp-p/wasi-build to main September 25, 2024 10:41
@samwillis samwillis force-pushed the samwillis/interactive-one-from-file branch 3 times, most recently from ebdd8af to 9fbeae8 Compare October 7, 2024 18:33
+#define PGS_IN "/tmp/pglite/base/.s.PGSQL.5432.in"
+#define PGS_OLOCK "/tmp/pglite/base/.s.PGSQL.5432.lock.out"
+#define PGS_OUT "/tmp/pglite/base/.s.PGSQL.5432.out"
+#define PGS_ILOCK "/tmp/pglite/.s.PGSQL.5432.lock.in"
Copy link
Collaborator

@pmp-p pmp-p Oct 8, 2024

Choose a reason for hiding this comment

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

potential problem is when mounting a db path under node running native, it is the only directory shared with host : /tmp/pglite is only MEMFS so other processes cannot use the socket file (eg to make a unix socket gateway ). /tmp/pglite/base is expected to be in raw mounted fs

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 move them as it doesn't seem to be possible to mount a device inside /tmp/pglite/base with the NodeFS, it throws a FS error.

@samwillis samwillis force-pushed the samwillis/interactive-one-from-file branch 3 times, most recently from b9e3b91 to 409c7fa Compare October 8, 2024 16:44
@samwillis samwillis force-pushed the samwillis/interactive-one-from-file branch from 7ab2382 to 2553edc Compare October 8, 2024 19:09
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