Skip to content

[GLUTEN-9199][VL] Fix error when creating shuffle file: open with O_CREAT or O_TMPFILE in second argument needs 3 arguments #9200

Merged
marin-ma merged 1 commit into
apache:mainfrom
zhztheplayer:wip-write-fd
Apr 2, 2025
Merged

[GLUTEN-9199][VL] Fix error when creating shuffle file: open with O_CREAT or O_TMPFILE in second argument needs 3 arguments #9200
marin-ma merged 1 commit into
apache:mainfrom
zhztheplayer:wip-write-fd

Conversation

@zhztheplayer

Copy link
Copy Markdown
Member

Fixes #9199 by setting a default mode to the open call.

@github-actions github-actions Bot added the VELOX label Apr 1, 2025
@github-actions

github-actions Bot commented Apr 1, 2025

Copy link
Copy Markdown
@zhztheplayer

Copy link
Copy Markdown
Member Author

@wangyum Would you help review the change? Thanks.

@wangyum wangyum left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@zhztheplayer

Copy link
Copy Markdown
Member Author
@marin-ma marin-ma merged commit c1cf957 into apache:main Apr 2, 2025
weiting-chen pushed a commit to weiting-chen/gluten that referenced this pull request Apr 26, 2025
…CREAT or O_TMPFILE in second argument needs 3 arguments (apache#9200)
philo-he added a commit that referenced this pull request Apr 29, 2025
* [GLUTEN-9199][VL] Fix error when creating shuffle file: open with O_CREAT or O_TMPFILE in second argument needs 3 arguments (#9200)

* [DOC] Fix broken links in documents (#9320)

* [GLUTEN-9369][DOC] Fix commands in the Gluten C++ debugging developer doc (#9368)

* [VL][CI] Change to use JDK-17 for Spark 3.3/3.4/3.5 tests (#9209)

* [VL][CI] Bump to use ubuntu-22.04 runner (#9262)

---------

Co-authored-by: Hongze Zhang <hongze.zzz123@gmail.com>
Co-authored-by: PHILO-HE <philo@apache.org>
Co-authored-by: Dina Suehiro Jones <dina.s.jones@intel.com>
Co-authored-by: Yuan <yuanzhou@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 participants