Skip to content

Conversation

davecramer
Copy link
Member

@vlsi @sehrope
This was done with some massaging but mostly using AI
Thoughts ?

CHANGELOG.md Outdated
Comment on lines 16 to 17
• test: Added CI executions with adaptive_fetch=true by default for performance testing [PR #bd572f40](https://github.com/pgjdbc/pgjdbc/commit/bd572f40)
• test: Added tests with reWriteBatchedInserts=true [PR #972cddd9](https://github.com/pgjdbc/pgjdbc/commit/972cddd9)
Copy link
Member

Choose a reason for hiding this comment

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

Those do not seem to belong to performance improvements section

CHANGELOG.md Outdated
## Bug Fixes

### Protocol & Connection Handling
• fix: Fixed sending extra_float_digits [PR #3491](https://github.com/pgjdbc/pgjdbc/pull/3491)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
• fix: Fixed sending extra_float_digits [PR #3491](https://github.com/pgjdbc/pgjdbc/pull/3491)
• fix: send extra_float_digits=3 for PostgreSQL 12+ as well [PR #3491](https://github.com/pgjdbc/pgjdbc/pull/3491)

Copy link
Member Author

Choose a reason for hiding this comment

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

actually we don't send extra_float_digits for 12+. After 12 we don't send it

Copy link
Member

@vlsi vlsi May 6, 2025

Choose a reason for hiding this comment

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

Why don't we send extra_float_digits for after 12?

Copy link
Member Author

Choose a reason for hiding this comment

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

@davecramer davecramer changed the title new changelog format Prepare release notes for release 42_7_6 (new format) May 27, 2025
@davecramer davecramer requested a review from sehrope May 27, 2025 18:54
@davecramer
Copy link
Member Author

@vlsi @sehrope any issues before I push and release ?

@sehrope
Copy link
Member

sehrope commented May 27, 2025

One possible movement to another section. Otherwise looks good.

@davecramer davecramer merged commit 689708f into pgjdbc:master May 28, 2025
16 of 17 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.

3 participants