Skip to content

fix: Bump folly tag to v2026.02.16.00#17695

Closed
sungwoo-XCENA wants to merge 3 commits into
facebookincubator:mainfrom
xflare-ai:bump-folly-version
Closed

fix: Bump folly tag to v2026.02.16.00#17695
sungwoo-XCENA wants to merge 3 commits into
facebookincubator:mainfrom
xflare-ai:bump-folly-version

Conversation

@sungwoo-XCENA
Copy link
Copy Markdown
Contributor

This fixes #17694.

In summary, AsyncIoContext.cpp conditionally included <folly/io/async/IoUringOptions.h>, which does not exist in the current pinned folly version. This fix bumps the folly version to v2026.02.16.00.

@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 2, 2026

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit 05cebd7
🔍 Latest deploy log https://app.netlify.com/projects/meta-velox/deploys/6a1e774109f7650008185539

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 2, 2026
@sungwoo-XCENA sungwoo-XCENA changed the title fix: bump folly tag to v2026.02.16.00 fix: Bump folly tag to v2026.02.16.00 Jun 2, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 2026

Build Impact Analysis

No build targets affected by this change.


Slow path • Graph generated from PR branch

sungwoo-XCENA and others added 2 commits June 2, 2026 06:03
The tag bump left VELOX_FOLLY_BUILD_SHA256_CHECKSUM at the v2026.01.05.00
hash, so the bundled-dependencies build failed folly download verification
with a hash mismatch.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
v2026.02.16.00 carries folly's IoUringOptions.h, but fizz at that tag
links a mis-named benchmark target (Folly::folly_benchmark, while folly
exports follybenchmark), breaking the fizz build. fizz fixes this at
v2026.06.01.00 -- the earliest FBOS tag where folly and fizz agree on
the benchmark target and folly still provides IoUringOptions.h.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@sungwoo-XCENA
Copy link
Copy Markdown
Contributor Author

Bumped folly version to v2026.06.01.00 because fizz's v2026.02.16.00 version had a mismatch problem with folly, which was fixed at v2026.06.01.00. That was the common denominator between this issue and fizz issue.

@czentgr
Copy link
Copy Markdown
Collaborator

czentgr commented Jun 3, 2026

As FYI this not only updates folly but the entire stack of the FBOS dependencies. This might introduce other problems.
We were planning on doing this eventually (after a thrift PR #16019 has merged) - we want to get change that is currently patched in that PR but risk of issues could cause to delay that PR even further.

You also need to update https://github.com/facebookincubator/velox/blob/main/CMake/resolve_dependency_modules/README.md
Edit: the commit and PR abstract should also reflect the version change made later.

@sungwoo-XCENA
Copy link
Copy Markdown
Contributor Author

@czentgr Thanks for the feedback. I opened this draft PR to run the github CI workflow if that version bump could fix the problem. But as you said, I guess it's more complicated than that. I will close this draft and wait for #16019 and other follow-ups to merge.

@sungwoo-XCENA sungwoo-XCENA deleted the bump-folly-version branch June 4, 2026 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build fails when liburing is installed with "IoUringOptions.h not found" message

2 participants