Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds changelog for 12.1.3 #7587

Merged
merged 7 commits into from Apr 22, 2024
Merged

Adds changelog for 12.1.3 #7587

merged 7 commits into from Apr 22, 2024

Conversation

gurkanindibay
Copy link
Contributor

No description provided.

@gurkanindibay gurkanindibay marked this pull request as ready for review April 18, 2024 09:10
Copy link

codecov bot commented Apr 18, 2024

Codecov Report

Merging #7587 (000bc09) into main (a0151aa) will increase coverage by 0.00%.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7587   +/-   ##
=======================================
  Coverage   89.69%   89.70%           
=======================================
  Files         283      283           
  Lines       60505    60506    +1     
  Branches     7538     7538           
=======================================
+ Hits        54270    54275    +5     
+ Misses       4081     4078    -3     
+ Partials     2154     2153    -1     

Copy link
Member

@onurctirtir onurctirtir left a comment

Choose a reason for hiding this comment

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

The following are missing changelog entries, could you work with @hanefi to come up with changelog entries for those PRs?

Let's also write down changelog entries you would come up with back to relevant PR descriptions so that in a future release we can re-use them without having to check changelog.md:

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@gurkanindibay
Copy link
Contributor Author

The following are missing changelog entries, could you work with @hanefi to come up with changelog entries for those PRs?

Let's also write down changelog entries you would come up with back to relevant PR descriptions so that in a future release we can re-use them without having to check changelog.md:

Added descriptions in those PR's and added those descriptions as the changelog item in the Changelog file. @hanefi @onurctirtir could you check if it is ok?

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@onurctirtir
Copy link
Member

(Sorry, I was out on Friday afternoon hence just had a chance to take a look into the PR)

Co-authored-by: Onur Tirtir <onurcantirtir@gmail.com>
Copy link
Member

@hanefi hanefi left a comment

Choose a reason for hiding this comment

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

Please check line lengths after applying my suggestions.

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated
* Fixes bug when hostname in pg_dist_node resolves to multiple IPs
(#7377)

* Fixes performance issue in when tracking foreign key constraints on
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
* Fixes performance issue in when tracking foreign key constraints on
* Fixes a performance issue in when tracking foreign key constraints on

Co-authored-by: Hanefi Onaldi <Hanefi.Onaldi@microsoft.com>
@gurkanindibay gurkanindibay merged commit 553d5ba into main Apr 22, 2024
126 checks passed
@gurkanindibay gurkanindibay deleted the changelog_12_1_3 branch April 22, 2024 12:38
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.

None yet

3 participants