Skip to content

Issue #956: Disable SQLAlchemy echo in MDR API (env-driven) - #1147

Open
dereck-symmetry wants to merge 1 commit into
mainfrom
issue-956---sqlalchemy-updates
Open

Issue #956: Disable SQLAlchemy echo in MDR API (env-driven)#1147
dereck-symmetry wants to merge 1 commit into
mainfrom
issue-956---sqlalchemy-updates

Conversation

@dereck-symmetry

Copy link
Copy Markdown

Include:

Related Issues

Closes #956

Type of Change
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality
    to not work as expected)
  • Documentation update
  • Infrastructure/deployment change
  • Performance improvement
  • Code refactoring
Project Area(s) Affected
  • bases/
  • components/
  • projects/
  • orchestrators/
  • frontends/
  • deployments/
  • cloudformation/ or sam/ templates
  • reference_data/
  • scripts/
  • test/ or e2e/
  • Database schema (migrations)
  • API endpoints
  • Documentation (docs/, READMEs, ARCHITECTURE.md, CLAUDE.md)
Checklist
  • commit message follows commit guidelines (see commitlint.config.mjs)
  • [X ] code passes linting checks (uv run ruff check)
  • code passes formatting checks (uv run ruff format)
  • code passes type checking (uv run ty check)
  • pre-commit hooks have been run successfully
  • configuration changes: relevant folder README updated
Testing
  • Manual testing performed
  • Automated tests added/updated
  • Integration testing completed
Additional Notes

Configuration updates done with the help of LLM. Did a complete deploy locally with docker to assure that the data was working.

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

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

MDR API SQLAlchemy engine uses echo=True — logs every SQL statement to CloudWatch

1 participant