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

refactor: Move planner outside the gql directory #907

Merged
merged 2 commits into from
Oct 19, 2022

Conversation

AndrewSisley
Copy link
Contributor

Relevant issue(s)

Resolves #904

Description

Moves planner and mapper packages outside the gql directory. No code has been changed.

@AndrewSisley AndrewSisley added refactor This issue specific to or requires *notable* refactoring of existing codebases and components action/no-benchmark Skips the action that runs the benchmark. area/mapper Related to the mapper components area/planner Related to the planner system labels Oct 19, 2022
@AndrewSisley AndrewSisley added this to the DefraDB v0.4 milestone Oct 19, 2022
@AndrewSisley AndrewSisley requested a review from a team October 19, 2022 15:06
@AndrewSisley AndrewSisley self-assigned this Oct 19, 2022
@AndrewSisley AndrewSisley changed the title refactor: Move planner and mapper outside the gql directory refactor: Move planner outside the gql directory Oct 19, 2022
@AndrewSisley AndrewSisley changed the title refactor: Move planner outside the gql directory refactor: Move planner outside the gql directory Oct 19, 2022
Copy link
Collaborator

@fredcarle fredcarle left a comment

Choose a reason for hiding this comment

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

LGTM

@AndrewSisley AndrewSisley force-pushed the sisley/refactor/I904-mv-planner branch from dd42a6a to aefcba5 Compare October 19, 2022 16:52
@AndrewSisley AndrewSisley merged commit 574d853 into develop Oct 19, 2022
@AndrewSisley AndrewSisley deleted the sisley/refactor/I904-mv-planner branch October 19, 2022 17:12
shahzadlone pushed a commit to shahzadlone/defradb that referenced this pull request Feb 23, 2024
* Move planner out of the gql dir

* Move mapper out of the gql dir
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action/no-benchmark Skips the action that runs the benchmark. area/mapper Related to the mapper components area/planner Related to the planner system refactor This issue specific to or requires *notable* refactoring of existing codebases and components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GQL Rework: Move planner outside of the gql directory
2 participants