Uh oh!
There was an error while loading. Please reload this page.
GH-43576: [Java] Gandiva Tests are failing due to linking issues - #43978
Conversation
vibhatha
commented
Sep 9, 2024
@lidavidm There are a very high number of test cases failing in the disabled test class. Instead of disabling each one, I disabled the class for now, but created a Github issue and tagged it in the disable message. |
danepitkin
left a comment
There was a problem hiding this comment.
Thank you, I think this is the right move. Hopefully Gandiva maintainers can help repair this module.
vibhatha
commented
Sep 10, 2024
@danepitkin Yes, we need the Gandiva maintainers' support. |
After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit f3dd298. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. It also includes information about 101 possible false positives for unstable benchmarks that are known to sometimes produce them. |
…che#43978) ### Rationale for this change Gandiva tests are failing due to a linking issue and it is failing the Java CIs. But for most of the made PRs, we cannot verify the overall workflow given that those PRs are independent of the Gandiva component. ### What changes are included in this PR? This PR disables such failing tests temporarily such that once the Gandiva issue is fixed, re-enabling the tests. Re-enabling task will be tracked using https://github.com/apache/arrow/issues/43981 ### Are these changes tested? Yes, by existing CIs and tests. ### Are there any user-facing changes? No * GitHub Issue: #43576 Authored-by: Vibhatha Lakmal Abeykoon <vibhatha@gmail.com> Signed-off-by: David Li <li.davidm96@gmail.com>
…che#43978) ### Rationale for this change Gandiva tests are failing due to a linking issue and it is failing the Java CIs. But for most of the made PRs, we cannot verify the overall workflow given that those PRs are independent of the Gandiva component. ### What changes are included in this PR? This PR disables such failing tests temporarily such that once the Gandiva issue is fixed, re-enabling the tests. Re-enabling task will be tracked using https://github.com/apache/arrow/issues/43981 ### Are these changes tested? Yes, by existing CIs and tests. ### Are there any user-facing changes? No * GitHub Issue: #43576 Authored-by: Vibhatha Lakmal Abeykoon <vibhatha@gmail.com> Signed-off-by: David Li <li.davidm96@gmail.com>
Rationale for this change
Gandiva tests are failing due to a linking issue and it is failing the Java CIs. But for most of the made PRs, we cannot verify the overall workflow given that those PRs are independent of the Gandiva component.
What changes are included in this PR?
This PR disables such failing tests temporarily such that once the Gandiva issue is fixed, re-enabling the tests.
Re-enabling task will be tracked using https://github.com/apache/arrow/issues/43981
Are these changes tested?
Yes, by existing CIs and tests.
Are there any user-facing changes?
No