Skip to content

fix: replaced deprecated lazy=True with lazy='select' in SQLAlchemy m… #536

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

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

Conversation

ishansurdi
Copy link

…odels

Reviewer:
Estimate:


Ticket

Fixes: JIRA_TICKET_LINK

Description

Goal

Replace deprecated lazy=True in SQLAlchemy relationships with lazy="select" as per SQLAlchemy 2.x migration guidelines.

Changes

  • Replaced all instances of lazy=True with lazy="select" in models.
  • Ensured code compatibility with SQLAlchemy 2.x

Future Tasks (optional)

  • Monitor for any side effects in relationship loading.
  • Upgrade other deprecated SQLAlchemy APIs if present.

How has this been tested?

  • Verified that relationship loading behavior remains unchanged.

To-do before merge (optional)

  • Get confirmation from reviewer that no eager/lazy loading assumptions are broken.

Checklist

Fill with x for completed.

  • [ x] My code follows the style guidelines of this project
  • [ x] I have reviewed my own code to ensure good quality
  • I have tested the functionality of my code to ensure it works as intended
  • I have resolved merge conflicts

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.

1 participant