(calcite) branch main updated: [CALCITE-6390] Exclude Arrow project on Windows builds

[snuyanzin]

This is an automated email from the ASF dual-hosted git repository.

snuyanzin pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/calcite.git


The following commit(s) were added to refs/heads/main by this push:
     new 1506857f40 [CALCITE-6390] Exclude Arrow project on Windows builds
1506857f40 is described below

commit 1506857f404037b63dfd8a11880393b767bd1544
Author: Stamatis Zampetakis <zabe...@gmail.com>
AuthorDate: Mon Apr 29 11:26:25 2024 +0200

    [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 3a9ae23fce..fb932aac89 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 402ad4f03d..ecbe01e293 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<JavaPlugin> {
     tasks {
         configureEach<Test> {
+            // 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")
         }