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.

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

Set up Project Pages for Removal of SQLAlchemy #89

Merged
merged 5 commits into from
Mar 23, 2023
Merged

Set up Project Pages for Removal of SQLAlchemy #89

merged 5 commits into from
Mar 23, 2023

Conversation

mathemancer
Copy link
Contributor

@mathemancer mathemancer commented Mar 22, 2023 •
edited
Loading

Split into the following projects:

  • Move DDL operations to SQL functions
  • Move DML operations to SQL functions
  • Rewrite DQL using combination of SQL layer functions and as-yet-to-be-determined query writing framework
  • Complete removal of SQLAlchemy, adding any needed D3L (Data definition description language) functions for use by API

@mathemancer mathemancer marked this pull request as draft March 22, 2023 15:12
@mathemancer mathemancer marked this pull request as ready for review March 23, 2023 08:27
@mathemancer
Copy link
Contributor Author

@seancolsen I just put you as a retroactive reviewer for info purposes, and so you can reorganize in the wiki if you want.

@mathemancer mathemancer merged commit 225714e into master Mar 23, 2023
@mathemancer mathemancer deleted the removing_sql_alchemy branch March 23, 2023 08:28
@kgodey
Copy link
Contributor

kgodey commented Mar 27, 2023

@mathemancer Will you send an email with this plan for review?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seancolsen seancolsen Awaiting requested review from seancolsen

@dmos62 dmos62 Awaiting requested review from dmos62

Assignees
No one assigned
Labels
None yet
Projects
No open projects
[NO LONGER USED] Mathesar work tracker
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mathemancer @kgodey