Skip to content

[GLUTEN-9366][VL] Support Iceberg functions#10285

Merged
jinchengchenghh merged 2 commits intoapache:mainfrom
jinchengchenghh:iceberg_functions
Sep 9, 2025
Merged

[GLUTEN-9366][VL] Support Iceberg functions#10285
jinchengchenghh merged 2 commits intoapache:mainfrom
jinchengchenghh:iceberg_functions

Conversation

@jinchengchenghh
Copy link
Contributor

@jinchengchenghh jinchengchenghh commented Jul 30, 2025

Only bucket function is supported, other functions are fallback now, all the function implementation are in facebookincubator/velox#14276

@github-actions github-actions bot added the CORE works for Gluten Core label Jul 30, 2025
@github-actions
Copy link

#9366

@github-actions
Copy link

Run Gluten Clickhouse CI on x86

Copy link
Member

@zhztheplayer zhztheplayer left a comment

Choose a reason for hiding this comment

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

👍 Thanks!

Comment on lines +211 to +210
case i @ StaticInvoke(_, _, "invoke", _, _, _, _, _) =>
return replaceIcebergStaticInvoke(i, attributeSeq, expressionsMap)
Copy link
Member

Choose a reason for hiding this comment

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

It looks like more accurate to match on org.apache.iceberg.spark.functions.* than on invoke to forward to a method as replaceIcebergStaticInvoke?

@github-actions
Copy link

github-actions bot commented Aug 5, 2025

Run Gluten Clickhouse CI on x86

@github-actions
Copy link

Run Gluten Clickhouse CI on x86

@github-actions
Copy link

github-actions bot commented Sep 8, 2025

Run Gluten Clickhouse CI on x86

1 similar comment
@github-actions
Copy link

github-actions bot commented Sep 8, 2025

Run Gluten Clickhouse CI on x86

@github-actions
Copy link

github-actions bot commented Sep 8, 2025

Run Gluten Clickhouse CI on x86

@jinchengchenghh jinchengchenghh merged commit 1c33b26 into apache:main Sep 9, 2025
101 of 102 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CORE works for Gluten Core VELOX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants