Skip to content

Graph node dev mode #5982

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 6 commits into
base: krishna/file-link-resolver
Choose a base branch
from
Open

Graph node dev mode #5982

wants to merge 6 commits into from

Conversation

incrypto32
Copy link
Member

@incrypto32 incrypto32 commented May 1, 2025 •
edited
Loading

  • Uses File based link resolver to run subgraphs from file system
  • Listen for file system events to remove and redeploy subgraphs during development
  • Uses pg_temp to spin up temporary databases in build dirs of subgraphs

Follow up for #5977

@incrypto32 incrypto32 force-pushed the krishna/graph-dev branch from 2783041 to a414de1 Compare May 1, 2025 12:59
@incrypto32 incrypto32 changed the base branch from master to krishna/file-link-resolver May 1, 2025 13:02
@incrypto32 incrypto32 force-pushed the krishna/graph-dev branch from a414de1 to ff19f99 Compare May 1, 2025 13:55
@incrypto32 incrypto32 self-assigned this May 1, 2025
@lutter lutter self-requested a review May 1, 2025 15:54
@incrypto32 incrypto32 force-pushed the krishna/file-link-resolver branch from 3ef16d5 to 23e91f1 Compare May 6, 2025 12:59
@incrypto32 incrypto32 force-pushed the krishna/graph-dev branch from ff19f99 to e5a9e23 Compare May 6, 2025 13:02
lutter
lutter approved these changes May 8, 2025
View reviewed changes
Copy link
Collaborator

@lutter lutter left a comment

Choose a reason for hiding this comment

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

This all looks great! There are a few improvements that would streamline the code, but this is a really good addition

@incrypto32 incrypto32 force-pushed the krishna/file-link-resolver branch 5 times, most recently from f7e1520 to cd2d014 Compare May 9, 2025 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lutter lutter lutter approved these changes

Assignees

@incrypto32 incrypto32

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@incrypto32 @lutter