Skip to content

fix: Remove nodecomposite warning logs when using dawgs graph driver BED-8293#2798

Merged
stephanieslamb merged 13 commits into
stage/v9.2.0from
BED-8293
May 22, 2026
Merged

fix: Remove nodecomposite warning logs when using dawgs graph driver BED-8293#2798
stephanieslamb merged 13 commits into
stage/v9.2.0from
BED-8293

Conversation

@stephanieslamb
Copy link
Copy Markdown
Contributor

@stephanieslamb stephanieslamb commented May 20, 2026

Description

Replaced pg.NewPool with standard pgxpool.New in OpenDatabase() since the relational database doesn't need graph-specific type handling

Motivation and Context

Resolves BED-8293

Why is this change required? What problem does it solve?
BHCE API logs were flooded with nodecomposite does not exist and edgecomposite does not exist errors. Updated the connection pool type that checks for these graph types.

How Has This Been Tested?

Existing tests run. Application runs as expected. Logs are no longer flooded with nodecomposite and edgecomposite errors

Screenshots (optional):

Screenshot 2026-05-20 at 10 50 39 AM

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

@stephanieslamb stephanieslamb self-assigned this May 20, 2026
@stephanieslamb stephanieslamb added the api A pull request containing changes affecting the API code. label May 20, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 20, 2026 •
edited
Loading

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: b895ebb2-d7e9-407a-b2d4-c2d3ae684144

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch BED-8293

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@stephanieslamb stephanieslamb changed the title Bed 8293 fix: Remove nodecomposite warning logs when using dawgs graph driver BED-8293 May 20, 2026
@stephanieslamb stephanieslamb marked this pull request as draft May 20, 2026 18:50
@stephanieslamb stephanieslamb marked this pull request as ready for review May 21, 2026 20:39
zinic
zinic approved these changes May 22, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@zinic zinic left a comment

Choose a reason for hiding this comment

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

:shipit:

@stephanieslamb stephanieslamb merged commit 905eb1d into stage/v9.2.0 May 22, 2026
12 checks passed
@stephanieslamb stephanieslamb deleted the BED-8293 branch May 22, 2026 21:17
@github-actions github-actions Bot locked and limited conversation to collaborators May 22, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@zinic zinic zinic approved these changes

Assignees

@stephanieslamb stephanieslamb

Labels

api A pull request containing changes affecting the API code.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@stephanieslamb @zinic @superlinkx