Skip to content

Core: Expose table incremental scan for appends API in SerializableTable#10682

Merged
nastra merged 3 commits into
apache:mainfrom
deniskuzZ:incremental_mv
Jul 12, 2024
Merged

Core: Expose table incremental scan for appends API in SerializableTable#10682
nastra merged 3 commits into
apache:mainfrom
deniskuzZ:incremental_mv

Conversation

@deniskuzZ
Copy link
Copy Markdown
Member

@deniskuzZ deniskuzZ commented Jul 11, 2024

required for HIVE-27101: Support incremental materialized view rebuild when Iceberg source tables have insert operation only

@github-actions github-actions Bot added the core label Jul 11, 2024
@nastra
Copy link
Copy Markdown
Contributor

nastra commented Jul 11, 2024

@deniskuzZ can you add the missing @Override please?

return lazyTable().newScan();
}

public IncrementalAppendScan newIncrementalAppendScan() {
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.

can you also override newIncrementalChangelogScan() here please?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

thanks @nastra for the review! added missing @Override annotation

Copy link
Copy Markdown
Member Author

@deniskuzZ deniskuzZ Jul 11, 2024

Choose a reason for hiding this comment

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

+ newIncrementalChangelogScan() override

@nastra nastra merged commit 07002fa into apache:main Jul 12, 2024
@deniskuzZ deniskuzZ deleted the incremental_mv branch July 13, 2024 08:42
jasonf20 pushed a commit to jasonf20/iceberg that referenced this pull request Aug 4, 2024
zachdisc pushed a commit to zachdisc/iceberg that referenced this pull request Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants