Skip to content

[VL][branch-1.7] Backport: Fix build issues reported by weekly jobs (#12321)#12600

Merged
philo-he merged 4 commits into
apache:branch-1.7from
philo-he:backport-build-fix
Jul 23, 2026
Merged

[VL][branch-1.7] Backport: Fix build issues reported by weekly jobs (#12321)#12600
philo-he merged 4 commits into
apache:branch-1.7from
philo-he:backport-build-fix

Conversation

@philo-he

@philo-he philo-he commented Jul 22, 2026

Copy link
Copy Markdown
Member

What changes are proposed in this pull request?

Backport three PRs. The last two PRs are also required for fixing CI failures.

Yuan helped fix another issue related to unexpectedly building Arrow in Velox setup script: #12611

How was this patch tested?

CI.

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

No.

Co-authored-by: Yuan <yuanzhou@apache.org>
@philo-he

Copy link
Copy Markdown
Member Author

Ubuntu weekly build failed:

CMake Error at cmake_modules/BuildUtils.cmake:297 (target_link_libraries):
  Target "arrow_testing_objlib" links to:

    Boost::process

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

Call Stack (most recent call first):
  src/arrow/CMakeLists.txt:1124 (add_arrow_lib)

@zhouyuan, do you happen to know the reason? I remember the main branch has a similar issue before your fix.

@zhouyuan

Copy link
Copy Markdown
Member

@philo-he IIRC the issue is from velox side, in their setup script will try to build arrow but cmake file is not correct, in my fix I was using gluten local build arrow script

@philo-he

philo-he commented Jul 23, 2026

Copy link
Copy Markdown
Member Author

@philo-he IIRC the issue is from velox side, in their setup script will try to build arrow but cmake file is not correct, in my fix I was using gluten local build arrow script

Thank you so much for the fix! Do we also need to fix the main branch to replace setup-centos9.sh with setup-ubuntu.sh? If so, could you also help fix it in main branch? Thanks!

sed -i "s|run_and_time install_arrow||g" scripts/setup-centos9.sh

@philo-he
philo-he force-pushed the backport-build-fix branch from d8536fc to 268f9f3 Compare July 23, 2026 18:50
@philo-he
philo-he merged commit 4f2bf04 into apache:branch-1.7 Jul 23, 2026
17 checks passed
@philo-he
philo-he deleted the backport-build-fix branch July 23, 2026 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants