From 1506857f404037b63dfd8a11880393b767bd1544 Mon Sep 17 00:00:00 2001 From: Stamatis Zampetakis Date: Mon, 29 Apr 2024 11:26:25 +0200 Subject: [PATCH] [CALCITE-6390] Exclude Arrow project on Windows builds Close apache/calcite#3778 --- .github/workflows/main.yml | 8 ++------ arrow/build.gradle.kts | 6 ++++++ 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml index 3a9ae23fce0d..fb932aac8933 100644 --- a/.github/workflows/main.yml +++ b/.github/workflows/main.yml @@ -69,9 +69,7 @@ jobs: with: job-id: jdk${{ matrix.jdk }} remote-build-cache-proxy-enabled: false - # Arrow build is excluded because it is not supported on Windows - # See https://arrow.apache.org/docs/java/install.html#system-compatibility - arguments: --scan --no-parallel --no-daemon build javadoc --exclude-task :arrow:build + arguments: --scan --no-parallel --no-daemon build javadoc - name: 'sqlline and sqllsh' shell: cmd run: | @@ -105,9 +103,7 @@ jobs: with: job-id: jdk${{ matrix.jdk }} remote-build-cache-proxy-enabled: false - # Arrow build is excluded because it is not supported on Windows - # See https://arrow.apache.org/docs/java/install.html#system-compatibility - arguments: --scan --no-parallel --no-daemon build --exclude-task :arrow:build + arguments: --scan --no-parallel --no-daemon build - name: 'sqlline and sqllsh' shell: cmd run: | diff --git a/arrow/build.gradle.kts b/arrow/build.gradle.kts index 402ad4f03d32..ecbe01e2932d 100644 --- a/arrow/build.gradle.kts +++ b/arrow/build.gradle.kts @@ -15,6 +15,7 @@ * limitations under the License. */ import com.github.vlsi.gradle.dsl.configureEach +import org.gradle.internal.os.OperatingSystem dependencies { api(project(":core")) @@ -36,6 +37,11 @@ dependencies { plugins.withType { tasks { configureEach { + // Tests disabled on Windows due to failures. Arrow is not tested on Windows: + // https://arrow.apache.org/docs/java/install.html#system-compatibility + enabled = !OperatingSystem.current().isWindows + // The following JVM options are required when using certain JDKs >= 9 + // https://arrow.apache.org/docs/java/install.html#java-compatibility jvmArgs("-XX:+IgnoreUnrecognizedVMOptions") jvmArgs("--add-opens=java.base/java.nio=ALL-UNNAMED") }