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

Remove LazyBlock #25255

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Remove LazyBlock #25255

wants to merge 4 commits into from

Conversation

dain
Copy link
Member

@dain dain commented Mar 7, 2025

Description

With the new SourcePage interface lazy blocks are not needed anymore.

Release notes

( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
(X) Release notes are required, with the following suggested text:

## SPI
* Remove LazyBlock. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Mar 7, 2025
@github-actions github-actions bot added delta-lake Delta Lake connector hive Hive connector labels Mar 7, 2025
@dain dain force-pushed the remove-lazy-block branch from 27154e3 to a120570 Compare March 7, 2025 23:24
@starburstdata-automation
Copy link

starburstdata-automation commented Mar 18, 2025

Started benchmark workflow for this PR with test type = iceberg/sf1000_parquet_part.

Building Trino finished with status: success
Benchmark finished with status: success
Comparing results to the static baseline values, follow above workflow link for more details/logs.
Status message: NO Regression found.
Benchmark Comparison to the closest run from Master: Report

@starburstdata-automation
Copy link

starburstdata-automation commented Mar 18, 2025

Started benchmark workflow for this PR with test type = iceberg/sf1000_parquet_unpart.

Building Trino finished with status: success
Benchmark finished with status: success
Comparing results to the static baseline values, follow above workflow link for more details/logs.
Status message: NO Regression found.
Benchmark Comparison to the closest run from Master: Report

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed delta-lake Delta Lake connector hive Hive connector
Development

Successfully merging this pull request may close these issues.

3 participants