Skip to content

Changelog entries for 12.1.0 - #7194

Merged
aykut-bozkurt merged 13 commits into
mainfrom
citus-12.1.0-changelog-2023-8-12-11-53-12
Sep 13, 2023
Merged

Changelog entries for 12.1.0#7194
aykut-bozkurt merged 13 commits into
mainfrom
citus-12.1.0-changelog-2023-8-12-11-53-12

Conversation

@aykut-bozkurt

@aykut-bozkurt aykut-bozkurt commented Sep 12, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@naisila

Naisila Puka (naisila) commented Sep 12, 2023

Copy link
Copy Markdown
Contributor

I added the following changelog items for PG16 - basically one entry for PG16RC1 support, and separate entries for PG16 new features that are supported/propagated in Citus:

* Adds PostgreSQL 16 RC1 support (#7173)

* Propagates BUFFER_USAGE_LIMIT option in VACUUM and ANALYZE (#7114)

* Propagates PROCESS_MAIN, SKIP_DATABASE_STATS, ONLY_DATABASE_STATS options in VACUUM (#7114)

* Propagates GENERIC_PLAN option in EXPLAIN (#7141)

* Adds citus_truncate_trigger to Citus foreign tables (#7170)

* Propagates "rules" option in CREATE COLLATION (#7185)

Önder Kalacı (@onderkalaci) do you agree on these entries? I am wondering if the main entry should be Adds PostgreSQL 16 support and omit RC1 since we expect GA this week and it will be before 12.1 release anyway

tejeswarm
Teja Mupparti (tejeswarm) reviewed Sep 12, 2023
View reviewed changes
Comment thread CHANGELOG.md Outdated
* Fixes PROCESS_TOAST default value to true (#7122)

PR #7081 : Improvise MERGE ON clause datatype mismatch error for brevity.
* Provide better error messaging for MERGE ON clause, datatype mismatc (todo: Teja)

@tejeswarm Teja Mupparti (tejeswarm) Sep 12, 2023

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The above line 44 looks good to me

@aykut-bozkurt
aykut-bozkurt marked this pull request as ready for review September 12, 2023 17:27
@codecov

codecov Bot commented Sep 13, 2023 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #7194 (3599f2d) into main (e0683aa) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #7194      +/-   ##
==========================================
- Coverage   93.22%   93.21%   -0.01%     
==========================================
  Files         274      274              
  Lines       59435    59435              
==========================================
- Hits        55407    55403       -4     
- Misses       4028     4032       +4

onurctirtir
Onur Tirtir (onurctirtir) approved these changes Sep 13, 2023
View reviewed changes

@onurctirtir Onur Tirtir (onurctirtir) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I pushed my suggested changes to the same branch.

Before merging, can we come up with a more user facing description instead of this?:

  • Adds citus_truncate_trigger to Citus foreign tables (#7170)

onurctirtir
Onur Tirtir (onurctirtir) reviewed Sep 13, 2023
View reviewed changes
Comment thread CHANGELOG.md Outdated
onurctirtir
Onur Tirtir (onurctirtir) reviewed Sep 13, 2023
View reviewed changes
Comment thread CHANGELOG.md Outdated
@onurctirtir

Onur Tirtir (onurctirtir) commented Sep 13, 2023

Copy link
Copy Markdown
Member

I added the following changelog items for PG16 - basically one entry for PG16RC1 support, and separate entries for PG16 new features that are supported/propagated in Citus:

* Adds PostgreSQL 16 RC1 support (#7173)

* Propagates BUFFER_USAGE_LIMIT option in VACUUM and ANALYZE (#7114)

* Propagates PROCESS_MAIN, SKIP_DATABASE_STATS, ONLY_DATABASE_STATS options in VACUUM (#7114)

* Propagates GENERIC_PLAN option in EXPLAIN (#7141)

* Adds citus_truncate_trigger to Citus foreign tables (#7170)

* Propagates "rules" option in CREATE COLLATION (#7185)

Önder Kalacı (@onderkalaci) do you agree on these entries? I am wondering if the main entry should be Adds PostgreSQL 16 support and omit RC1 since we expect GA this week and it will be before 12.1 release anyway

makes sense to me

onurctirtir
Onur Tirtir (onurctirtir) reviewed Sep 13, 2023
View reviewed changes
Comment thread CHANGELOG.md Outdated
onurctirtir
Onur Tirtir (onurctirtir) reviewed Sep 13, 2023
View reviewed changes
Comment thread CHANGELOG.md
onurctirtir
Onur Tirtir (onurctirtir) reviewed Sep 13, 2023
View reviewed changes
Comment thread CHANGELOG.md Outdated
@aykut-bozkurt
aykut-bozkurt merged commit 9eafd03 into main Sep 13, 2023
@aykut-bozkurt
aykut-bozkurt deleted the citus-12.1.0-changelog-2023-8-12-11-53-12 branch September 13, 2023 09:12
onderkalaci
Önder Kalacı (onderkalaci) reviewed Sep 13, 2023
View reviewed changes
Comment thread CHANGELOG.md
* Add `citus_schema_move()` function which moves tables within a
distributed schema to another node (#7180)

* Adds `citus_pause_node()` UDF that allows pausing the node with given id

@onderkalaci Önder Kalacı (onderkalaci) Sep 13, 2023

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ops, the name of the UDF is citus_pause_node_within_txn, we should fix the changelog: https://github.com/citusdata/citus/pull/7089/files#diff-3b79898c97cb0fb3169790554076d0c3ced79f6c9e0dfc2b37103fd7f0aba8ffR1383

Jodi-Ann Francis (francisjodi) pushed a commit that referenced this pull request Nov 13, 2023
@aykut-bozkurt @naisila
@onurctirtir @francisjodi
Changelog entries for 12.1.0 (#7194)
5e17e4f 
Co-authored-by: naisila <nicypp@gmail.com>
Co-authored-by: Onur Tirtir <onurcantirtir@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@onurctirtir Onur Tirtir (onurctirtir) onurctirtir approved these changes
@tejeswarm Teja Mupparti (tejeswarm) tejeswarm approved these changes
+2 more reviewers
@onderkalaci Önder Kalacı (onderkalaci) onderkalaci left review comments
@naisila Naisila Puka (naisila) naisila approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@aykut-bozkurt @naisila @onurctirtir @onderkalaci @tejeswarm