Skip to content

DO NOT MERGE: 28.0.0 release notes#15173

Closed
ektravel wants to merge 14 commits into
apache:masterfrom
ektravel:28.0-wip-release-notes
Closed

DO NOT MERGE: 28.0.0 release notes#15173
ektravel wants to merge 14 commits into
apache:masterfrom
ektravel:28.0-wip-release-notes

Conversation

@ektravel

Copy link
Copy Markdown
Contributor

Draft PR used to collaborate on the release notes so that we can use the comment/suggestion system.

Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md Outdated
Comment thread docs/do-not-merge.md
Comment thread docs/do-not-merge.md

# Upgrade notes and incompatible changes

## Upgrade notes

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 think we should do something a bit different than previous releases before we finally publish these release notes, where this section is about upgrading from Druid 27, and then we also should include the upgrade notices from the previous few releases as well, just copied in from earlier release notes. So that way we have all the information in one place to cover like a year worth of upgrades. I don't think this needs to be added yet, we can wait until the last minute before publishing, but I think it would be helpful since operators don't always upgrade every release

Comment thread docs/do-not-merge.md Outdated

## SQL planner improvements

Druid uses Apache Calcite for SQL planning and optimization. Starting in Druid 28.0.0, the Calcite version has been upgraded from 1.21 to 1.35. This upgrade brings in many bug fixes in SQL planning from Calcite. As part of the upgrade, the behavior of type inference for [dynamic parameters](#dynamic-parameters) and the [recommended syntax for UNNEST](#new-syntax-for-sql-unnest) have changed.

@clintropolis clintropolis Oct 31, 2023

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.

this reads kind of funny, the updated recommendation for unnest isn't directly related to the calcite upgrade afaik, rather it is just less likely to run into problems with complicated queries if you use the more verbose cross join syntax. maybe we should just drop this line since we go over dynamic parameter changes immediately after (or move the part about it being related to calcite upgrade into that section)

@ektravel ektravel Nov 1, 2023

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@clintropolis Since the updated recommendation for unnest isn't directly related to the calcite upgrade, I think we should move it to "SQL UNNEST and ingestion flattening".

@ektravel ektravel changed the title DO NOT MERGE - 28.0.0 WIP release notes 28.0.0 release notes Nov 2, 2023
@ektravel
ektravel changed the base branch from master to 28.0.0 November 2, 2023 22:00
@ektravel
ektravel changed the base branch from 28.0.0 to master November 2, 2023 22:01
@317brian 317brian changed the title 28.0.0 release notes DO NOT MERGE: 28.0.0 release notes Nov 2, 2023
@ektravel

ektravel commented Nov 2, 2023

Copy link
Copy Markdown
Contributor Author

Closing PR 15173 in favor of #15315.

@ektravel ektravel closed this Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants