-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GH-42005: [Java][Integration][CI] Fix ARROW_BUILD_ROOT Path to find pom.xml #42008
Conversation
|
I verified that the path to find the # from /Users/lama/workspace/arrow-new/dev/archery/archery/utils
# Here is the path that was referenced before this PR is merged.
ARROW_ROOT: /Users/lama/workspace/arrow-new
# from /Users/lama/workspace/arrow-new/dev/archery/archery/integration
ARROW_BUILD_ROOT: /Users/lama/workspace |
@raulcd Could you please execute the command as you mentioned? |
@github-actions crossbow submit verify-rc-source-integration-linux-ubuntu-20.04-amd64 |
Revision: 849707f Submitted crossbow builds: ursacomputing/crossbow @ actions-0e1aa06ee4
|
@kou Could you please let me know if there might have been another intention in the PR you wrote? From my review of the code, it seems that in this file, the path needs to go up 4 levels to correctly refer |
It seems that the UPDATE: I attached the error message. ################# FAILURES #################
FAILED TEST: primitive_no_batches C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: primitive C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: primitive_zerolength C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: primitive_large_offsets C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: null C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: null_trivial C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: decimal C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: datetime C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: duration C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: map C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: map_non_canonical C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: nested C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: recursive_nested C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: union C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: custom_metadata C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: dictionary C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: dictionary_unsigned C++ producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: nested JS producing, Go consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: recursive_nested JS producing, Go consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: union JS producing, Go consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: custom_metadata JS producing, Go consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: dictionary JS producing, Go consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: dictionary_unsigned JS producing, Go consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: nested_dictionary JS producing, Go consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: extension JS producing, Go consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: primitive_no_batches Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: primitive Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: primitive_zerolength Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: primitive_large_offsets Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: null Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: null_trivial Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: decimal Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: datetime Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: duration Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: map Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: map_non_canonical Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: nested Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: recursive_nested Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: union Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: custom_metadata Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: dictionary Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: dictionary_unsigned Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: nested_dictionary Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
FAILED TEST: extension Go producing, JS consuming
<class 'FileNotFoundError'>: [Errno 2] No such file or directory: '/js'
127 failures, 200 skips
Failed to verify release candidate. See /tmp/arrow-HEAD.hhs4k for details.
Error: docker run --rm -e VERIFY_VERSION= -e VERIFY_RC= -e TEST_DEFAULT=0 -e TEST_INTEGRATION=1 --shm-size 1073741824 -e CCACHE_COMPILERCHECK=content -e CCACHE_COMPRESS=1 -e CCACHE_COMPRESSLEVEL=6 -e CCACHE_DIR=/ccache -e CCACHE_MAXSIZE=1G -e CMAKE_GENERATOR=Ninja -e GITHUB_ACTIONS=true -e TEST_APT=0 -e TEST_YUM=0 -v /home/runner/work/crossbow/crossbow/arrow:/arrow -v ubuntu-ccache:/ccache apache/arrow-dev:amd64-ubuntu-20.04-verify-rc /bin/bash -c git config --global --add safe.directory /arrow &&
/arrow/dev/release/verify-release-candidate.sh ${VERIFY_VERSION} ${VERIFY_RC} exited with non-zero exit code 1
Error: Process completed with exit code 1.
|
@raulcd Could you please command one more? Also, I understand that PR was aimed at improving documentation. However, I'm not sure why it included updates to the integration paths for Java and JS. If these integration paths are not related to improving documentation, I think the code needs to be rolled back.
|
@github-actions crossbow submit verify-rc-source-integration-linux-ubuntu-20.04-amd64 |
Revision: e5a8205 Submitted crossbow builds: ursacomputing/crossbow @ actions-7777758971
|
I haven't tested locally but the verification script just uses
This is the relevant piece: https://github.com/apache/arrow/blob/main/dev/release/verify-release-candidate.sh#L986-L1013 |
Umm... the error message is changed. It seems that the
|
I've re-run the job in case it was a fluke, otherwise we might need to open a different issue for that test. |
I think we need to review it more to solve the problem. |
@github-actions crossbow submit preview-docs -g wheel verify-rc-source-integration* |
Revision: e5a8205 Submitted crossbow builds: ursacomputing/crossbow @ actions-e708b89fd9 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 41ae29e. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. It also includes information about 3 possible false positives for unstable benchmarks that are known to sometimes produce them. |
Rationale for this change
This PR aims to fix the issue where the integration tests are failing due to the missing
/java/pom.xml
file. It appears that the current code incorrectly determines the path toARROW_BUILD_ROOT
, leading to the failure in locating thepom.xml
file.What changes are included in this PR?
ARROW_BUILD_ROOT
path determination logic intester_java.py
to correctly reference the project root.Are these changes tested?
Maybe, Yes.
Are there any user-facing changes?
No.
/java/pom.xml
#42005