Skip to content

Comments

V1.0.1#37

Merged
clauspruefer merged 14 commits intomainfrom
v1.0.1rc
Feb 18, 2026
Merged

V1.0.1#37
clauspruefer merged 14 commits intomainfrom
v1.0.1rc

Conversation

@clauspruefer
Copy link
Owner

@clauspruefer clauspruefer commented Feb 18, 2026

Pull Request

Description

Version 1.0.1. Adds default values for query_timeout, session_tmp_buffer, ssl and connect_timeout.

The tests in example 01-logical-replication rely on these default values and will not work otherwise (this PR also is a bug fix).

Type of Change

  • New feature
  • Bug fix

Copilot AI review requested due to automatic review settings February 18, 2026 10:29
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds default values for four optional database configuration parameters (query_timeout, session_tmp_buffer, ssl, and connect_timeout) to make them truly optional. The changes update the version to 1.0.1 and simplify the README description to be more generic.

Changes:

  • Added default values for query_timeout (5000ms) and session_tmp_buffer (128MB) in the class initialization
  • Added default values for ssl ('disable') and connect_timeout (10 seconds) in the connection method
  • Updated version from "1.0" to "1.0.1" in pyproject.toml
  • Simplified README description from "Web- / Application Servers" to just "De-Multiplexer"

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.

File Description
src/pool.py Added default value checks for query_timeout, session_tmp_buffer, ssl, and connect_timeout parameters
pyproject.toml Bumped version from 1.0 to 1.0.1
README.md Simplified description and updated diagrams to use generic "Server Service.py" instead of "WebServer Service.py"

@clauspruefer clauspruefer changed the title V1.0.1rc V1.0.1 Feb 18, 2026
@clauspruefer
Copy link
Owner Author

@copilot: Please update the CHANGELOG.md according to the current changes.

Copy link
Contributor

Copilot AI commented Feb 18, 2026

@clauspruefer I've opened a new pull request, #38, to work on those changes. Once the pull request is ready, I'll request review from you.

@clauspruefer
Copy link
Owner Author

@copilot Please re-update CHANGELOG.md (add current changes).

Copy link
Contributor

Copilot AI commented Feb 18, 2026

@clauspruefer I've opened a new pull request, #39, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 2 commits February 18, 2026 11:43
Co-authored-by: clauspruefer <17313789+clauspruefer@users.noreply.github.com>
@clauspruefer clauspruefer merged commit 3d77281 into main Feb 18, 2026
2 checks passed
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.

2 participants