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

xiangfu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 400b603abd open javac modules for google-java-format (#8531)
400b603abd is described below

commit 400b603abd699895b2f1833490e012183200176e
Author: Richard Startin <rich...@startree.ai>
AuthorDate: Wed Apr 13 23:00:53 2022 +0100

    open javac modules for google-java-format (#8531)
---
 .github/workflows/pinot_compatibility_tests.yml | 10 ++++-
 .github/workflows/pinot_tests.yml               | 50 ++++++++++++++++++++++---
 2 files changed, 54 insertions(+), 6 deletions(-)

diff --git a/.github/workflows/pinot_compatibility_tests.yml 
b/.github/workflows/pinot_compatibility_tests.yml
index 059f1c8bfe..acb0b1974c 100644
--- a/.github/workflows/pinot_compatibility_tests.yml
+++ b/.github/workflows/pinot_compatibility_tests.yml
@@ -48,7 +48,15 @@ jobs:
           NEW_COMMIT: ${{ github.event.inputs.newCommit }}
           WORKING_DIR: /tmp/compatibility-verifier
           TEST_SUITE: ${{ matrix.test_suite }}
-          MAVEN_OPTS: -Xmx2G -DskipShade -DfailIfNoTests=false 
-Dmaven.wagon.httpconnectionManager.ttlSeconds=25 
-Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false 
-Dmaven.wagon.http.pool=false
+          MAVEN_OPTS: >
+            -Xmx2G -DskipShade -DfailIfNoTests=false 
-Dmaven.wagon.httpconnectionManager.ttlSeconds=25
+            -Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false 
-Dmaven.wagon.http.pool=false
+            -XX:+IgnoreUnrecognizedVMOptions
+            --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
         run: .github/workflows/scripts/.pinot_compatibility_verifier.sh
       - name: Archive artifacts into zip
         if: always()
diff --git a/.github/workflows/pinot_tests.yml 
b/.github/workflows/pinot_tests.yml
index 4093a8d336..90dc4443ae 100644
--- a/.github/workflows/pinot_tests.yml
+++ b/.github/workflows/pinot_tests.yml
@@ -56,7 +56,15 @@ jobs:
           java-version: 11
       - name: Linter Test
         env:
-          MAVEN_OPTS: -Xmx2G -DskipShade -DfailIfNoTests=false 
-Dmaven.wagon.httpconnectionManager.ttlSeconds=25 
-Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false 
-Dmaven.wagon.http.pool=false
+          MAVEN_OPTS: >
+            -Xmx2G -DskipShade -DfailIfNoTests=false 
-Dmaven.wagon.httpconnectionManager.ttlSeconds=25
+            -Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false 
-Dmaven.wagon.http.pool=false
+            -XX:+IgnoreUnrecognizedVMOptions
+            --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
         run: .github/workflows/scripts/.pinot_linter.sh
   unit-test:
     if: github.repository == 'apache/pinot'
@@ -82,7 +90,15 @@ jobs:
         env:
           RUN_INTEGRATION_TESTS: false
           RUN_TEST_SET: ${{ matrix.testset }}
-          MAVEN_OPTS: -Xmx2G -DskipShade -DfailIfNoTests=false 
-Dmaven.wagon.httpconnectionManager.ttlSeconds=25 
-Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false 
-Dmaven.wagon.http.pool=false
+          MAVEN_OPTS: >
+            -Xmx2G -DskipShade -DfailIfNoTests=false 
-Dmaven.wagon.httpconnectionManager.ttlSeconds=25
+            -Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false 
-Dmaven.wagon.http.pool=false
+            -XX:+IgnoreUnrecognizedVMOptions
+            --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
         run: .github/workflows/scripts/.pinot_test.sh
       - name: Upload coverage to Codecov
         run: |
@@ -112,7 +128,15 @@ jobs:
         env:
           RUN_INTEGRATION_TESTS: true
           RUN_TEST_SET: ${{ matrix.testset }}
-          MAVEN_OPTS: -Xmx2G -DskipShade -DfailIfNoTests=false 
-Dmaven.wagon.httpconnectionManager.ttlSeconds=25 
-Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false 
-Dmaven.wagon.http.pool=false
+          MAVEN_OPTS: >
+            -Xmx2G -DskipShade -DfailIfNoTests=false 
-Dmaven.wagon.httpconnectionManager.ttlSeconds=25
+            -Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false 
-Dmaven.wagon.http.pool=false
+            -XX:+IgnoreUnrecognizedVMOptions
+            --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
         run: .github/workflows/scripts/.pinot_test.sh
       - name: Upload coverage to Codecov
         run: |
@@ -144,7 +168,15 @@ jobs:
           OLD_COMMIT: ${{ matrix.old_commit }}
           WORKING_DIR: /tmp/compatibility-verifier
           TEST_SUITE: ${{ matrix.test_suite }}
-          MAVEN_OPTS: -Xmx2G -DskipShade -DfailIfNoTests=false 
-Dmaven.wagon.httpconnectionManager.ttlSeconds=25 
-Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false 
-Dmaven.wagon.http.pool=false
+          MAVEN_OPTS: >
+            -Xmx2G -DskipShade -DfailIfNoTests=false 
-Dmaven.wagon.httpconnectionManager.ttlSeconds=25
+            -Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false 
-Dmaven.wagon.http.pool=false
+            -XX:+IgnoreUnrecognizedVMOptions
+            --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
         run: .github/workflows/scripts/.pinot_compatibility_verifier.sh
 
   quickstarts:
@@ -182,7 +214,15 @@ jobs:
           java-version: 11
       - name: Build presto pinot driver with JDK 11
         env:
-          MAVEN_OPTS: -Xmx2G -DfailIfNoTests=false 
-Dmaven.wagon.httpconnectionManager.ttlSeconds=25 
-Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false 
-Dmaven.wagon.http.pool=false
+          MAVEN_OPTS: >
+            -Xmx2G -DfailIfNoTests=false 
-Dmaven.wagon.httpconnectionManager.ttlSeconds=25
+            -Dmaven.wagon.http.retryHandler.count=3 -Dhttp.keepAlive=false 
-Dmaven.wagon.http.pool=false
+            -XX:+IgnoreUnrecognizedVMOptions
+            --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED
+            --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
         run: |
           mvn clean install -DskipTests -Ppresto-driver -am -B -pl 
':presto-pinot-driver' -T 16 || exit 1
       - name: Set up JDK 8


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to