Skip to content

[GLUTEN-12339][FLINK] Bind native callback target for Gluten operators#12274

Merged
zhanglistar merged 6 commits into
apache:mainfrom
bigo-sg:feature/native-callback-bridge
Jun 26, 2026
Merged

[GLUTEN-12339][FLINK] Bind native callback target for Gluten operators#12274
zhanglistar merged 6 commits into
apache:mainfrom
bigo-sg:feature/native-callback-bridge

Conversation

@lgbo-ustc

@lgbo-ustc lgbo-ustc commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

What changes are proposed in this pull request?

Adds native callback target binding for Gluten Flink operators so Velox processing-time callbacks can schedule output draining through the operator mailbox.

How was this patch tested?

UTs

Was this patch authored or co-authored using generative AI tooling?

co-authored using generative AI

Related issue: #12339

dependency
bigo-sg/velox4j#33
bigo-sg/velox#36

@lgbo-ustc lgbo-ustc force-pushed the feature/native-callback-bridge branch from bb72cfe to 61e4db3 Compare June 17, 2026 03:21
@lgbo-ustc lgbo-ustc changed the title [WIP][FLINK] Bind native callback target for Gluten operators [GLUTEN-12339][FLINK] Bind native callback target for Gluten operators Jun 23, 2026
@lgbo-ustc lgbo-ustc marked this pull request as ready for review June 23, 2026 07:04

@KevinyhZou KevinyhZou left a comment

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.

LGTM

Comment thread .github/workflows/flink.yml Outdated
export folly_SOURCE=BUNDLED
git clone -b gluten-0530 https://github.com/bigo-sg/velox4j.git
cd velox4j && git reset --hard 115edf79d265a61c30d45dfcc6ce932ad92378ca
git clone -b feature/native-callback-bridge https://github.com/bigo-sg/velox4j.git

@zhanglistar zhanglistar Jun 23, 2026

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.

do not change this, just change following commit id is OK.

@lgbo-ustc lgbo-ustc Jun 23, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this will be updated when velox4j is merged. This is tmp for testing

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.

Yes, I know, if the branch is in remote, then we do not need this change, just modify commit id as the following line does.

@lgbo-ustc lgbo-ustc force-pushed the feature/native-callback-bridge branch from 9f890ae to e05a386 Compare June 24, 2026 07:21
@lgbo-ustc lgbo-ustc force-pushed the feature/native-callback-bridge branch from e05a386 to b882807 Compare June 26, 2026 02:01
@github-actions github-actions Bot removed the INFRA label Jun 26, 2026

@zhanglistar zhanglistar left a comment

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.

LGTM

@zhanglistar zhanglistar merged commit 5a5516b into apache:main Jun 26, 2026
4 checks passed
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.

3 participants