Skip to content

Feature/issue 90#121

Open
Vnajeb wants to merge 5 commits into
mainfrom
feature/issue-90
Open

Feature/issue 90#121
Vnajeb wants to merge 5 commits into
mainfrom
feature/issue-90

Conversation

@Vnajeb

@Vnajeb Vnajeb commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

Fixes #90

Changes proposed in this pull request:

  • Consistent logging hierarchy every module now uses logging.getLogger(__ name __), replacing bare print() calls and the misleading websockets.client logger name.
  • Connections, pool scaling, and query stages (prepare/execute/fetch/close) log at the appropriate level (INFO/DEBUG/ERROR).
  • Credential safe passwords and Kerberos tokens are never logged only host, realm, and error codes appear.

Before submitting

  • Change the base branch to dev if it is not already.
  • I've read and followed all steps in the Making a pull request
    section of the CONTRIBUTING docs.
  • I've updated or added any relevant docstrings, following the Google docstring style used
    elsewhere in the codebase.
  • If this PR fixes a bug, I've added a test that will fail without my fix.
  • If this PR adds a new feature, I've added tests that sufficiently cover my new functionality.

Najeb Abdullahi and others added 4 commits June 4, 2026 10:48
cursor.description, fetchone/fetchall/fetchmany now return proper
tuples, rowcount accurate, AsyncCursor parity, 24 unit tests added
…async_base_job, and add pool lifecycle logs
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.

Add structured logging across the codebase

1 participant