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

More page builder cleanup #23809

Merged
merged 1 commit into from
Oct 17, 2024
Merged

More page builder cleanup #23809

merged 1 commit into from
Oct 17, 2024

Conversation

dain
Copy link
Member

@dain dain commented Oct 16, 2024

Release notes

(x) Release notes are required, with the following suggested text:

## SPI
* Remove unnecessary `PageBuilder``newPageBuilderLike` method. ({issue}`issuenumber`)

@dain dain requested review from martint and wendigo October 16, 2024 21:15
@cla-bot cla-bot bot added the cla-signed label Oct 16, 2024
@github-actions github-actions bot added the hive Hive connector label Oct 16, 2024
core/trino-spi/pom.xml Outdated Show resolved Hide resolved
This method only has one usage and the existing reset method is better
@dain dain force-pushed the page-builder-cleanup branch from 7494e03 to 2747fa6 Compare October 17, 2024 01:15
@dain dain merged commit 8e070d6 into trinodb:master Oct 17, 2024
93 checks passed
@dain dain deleted the page-builder-cleanup branch October 17, 2024 05:53
@github-actions github-actions bot added this to the 463 milestone Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed hive Hive connector
Development

Successfully merging this pull request may close these issues.

2 participants