Skip to content

Druid 30.0.0 release notes follow-up#16509

Merged
317brian merged 4 commits intoapache:30.0.0from
ektravel:docs-30.0.0-patch1
May 30, 2024
Merged

Druid 30.0.0 release notes follow-up#16509
317brian merged 4 commits intoapache:30.0.0from
ektravel:docs-30.0.0-patch1

Conversation

@ektravel
Copy link
Copy Markdown
Contributor

This is a follow-up PR to address the comments on PR #16412

This PR has:

  • been self-reviewed.

Comment thread docs/release-info/release-notes.md Outdated
Copy link
Copy Markdown
Contributor

@LakshSingla LakshSingla left a comment

Choose a reason for hiding this comment

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

Thanks for the follow-up PR 🚀. The changes related to "Grouping on complex columns" LGTM.

Copy link
Copy Markdown
Contributor

@writer-jill writer-jill left a comment

Choose a reason for hiding this comment

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

Couple of suggestions for clarity

Comment thread docs/release-info/release-notes.md Outdated
Comment thread docs/release-info/release-notes.md Outdated
Comment thread docs/release-info/release-notes.md Outdated
Comment thread docs/release-info/release-notes.md Outdated
@ektravel ektravel requested a review from writer-jill May 29, 2024 16:14
Copy link
Copy Markdown
Contributor

@writer-jill writer-jill left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@317brian 317brian left a comment

Choose a reason for hiding this comment

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

Thanks for this cleanup!

@317brian 317brian merged commit afa8ae8 into apache:30.0.0 May 30, 2024
@317brian 317brian deleted the docs-30.0.0-patch1 branch May 30, 2024 17:03
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.

4 participants