Skip to content

Tvf analyzer final check#30

Open
mohsaka wants to merge 12 commits into
masterfrom
tvf_analyzer_final_check
Open

Tvf analyzer final check#30
mohsaka wants to merge 12 commits into
masterfrom
tvf_analyzer_final_check

Conversation

@mohsaka
Copy link
Copy Markdown
Owner

@mohsaka mohsaka commented Dec 14, 2025

Description

Motivation and Context

Impact

Test Plan

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.
  • If adding new dependencies, verified they have an OpenSSF Scorecard score of 5.0 or higher (or obtained explicit TSC approval for lower scores).

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== RELEASE NOTES ==

General Changes
* ... 
* ... 

Hive Connector Changes
* ... 
* ...

If release note is NOT required, use:

== NO RELEASE NOTE ==

mohsaka and others added 12 commits December 13, 2025 10:56
@mohsaka @kasiafi @xin-zhang2
Final analyzer/planner/optimizer changes for tvf except LocalExecutio…
80dca59 
…nPlanner and ExcludeColumns optimizer rule.

Co-authored-by: kasiafi <30203062+kasiafi@users.noreply.github.com>
Co-authored-by: Xin Zhang <desertsxin@gmail.com>
@xin-zhang2 @kasiafi @mohsaka
Add optimizer rules for table functions.
1e39489 
Co-authored-by: kasiafi <30203062+kasiafi@users.noreply.github.com>
Co-authored-by: mohsaka <135669458+mohsaka@users.noreply.github.com>
@xin-zhang2 @kasiafi @mohsaka
Support execution by operator for table functions
2af2563 
Co-authored-by: kasiafi <30203062+kasiafi@users.noreply.github.com>
Co-authored-by: mohsaka <135669458+mohsaka@users.noreply.github.com>
@mohsaka @kasiafi @xin-zhang2
Add table function exclude_columns
fe5ab94 
Changes adapted from trino/PR#16584
Author: kasiafi

Co-authored-by: kasiafi <30203062+kasiafi@users.noreply.github.com>
Co-authored-by: Xin Zhang <desertsxin@gmail.com>
@mohsaka @kasiafi
Add table function sequence
f33b4c6 
Changes adapted from trino/PR#16716
Author: kasiafi

Co-authored-by: kasiafi <30203062+kasiafi@users.noreply.github.com>
@mohsaka @kasiafi
Rewrite exclude_columns table function into projection
2b8142f 
Changes adapted from trino/PR#25493
Author: kasiafi

Co-authored-by: kasiafi <30203062+kasiafi@users.noreply.github.com>
@xin-zhang2 @mohsaka
Rename ImplementTableFunctionSource to TransformTableFunctionToTableF…
b164cba 
…unctionProcessor
@xin-zhang2 @mohsaka
fix testAllMethodsOverridden
d6f81b8
@xin-zhang2 @mohsaka
Address some review comments
8fe4d91
@xin-zhang2 @mohsaka
update PlanOptimizers
8170eaa
@mohsaka
Address comments
3e548ad
@mohsaka
Possible fix
80b403e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mohsaka @xin-zhang2