(calcite) branch main updated: [CALCITE-6484] Sonar analysis fails intermittently due to OOM

[asolimando]

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

asolimando 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 cc1d6b9310 [CALCITE-6484] Sonar analysis fails intermittently due to 
OOM
cc1d6b9310 is described below

commit cc1d6b9310760a529a1109bf3c94d6b61f6d076d
Author: Alessandro Solimando <alessandro.solima...@gmail.com>
AuthorDate: Thu Jul 18 15:25:25 2024 +0200

    [CALCITE-6484] Sonar analysis fails intermittently due to OOM
    
    - Increased JVM heap max memory to 1.5GB
    - Adopted systemProp.sonar.gradle.skipCompile=true and explicit build 
command
    - Replaced sonar.login with sonar.token
---
 Jenkinsfile       | 4 ++--
 gradle.properties | 5 +++--
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/Jenkinsfile b/Jenkinsfile
index 7ff37aba28..57fd6db08c 100644
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -44,9 +44,9 @@ node('ubuntu') {
       withEnv(["Path+JDK=$JAVA_JDK_17/bin","JAVA_HOME=$JAVA_JDK_17"]) {
         withCredentials([string(credentialsId: 'SONARCLOUD_TOKEN', variable: 
'SONAR_TOKEN')]) {
           if ( env.BRANCH_NAME.startsWith("PR-") ) {
-            sh './gradlew --no-parallel --no-daemon jacocoAggregateTestReport 
sonar -PenableJacoco -Porg.sonarqube.version=4.4.1.3373 
-Dsonar.pullrequest.branch=${CHANGE_BRANCH} 
-Dsonar.pullrequest.base=${CHANGE_TARGET} -Dsonar.pullrequest.key=${CHANGE_ID} 
-Dsonar.login=${SONAR_TOKEN}'
+            sh './gradlew build --no-parallel --no-daemon 
jacocoAggregateTestReport sonar -PenableJacoco 
-Porg.sonarqube.version=4.4.1.3373 -Dsonar.pullrequest.branch=${CHANGE_BRANCH} 
-Dsonar.pullrequest.base=${CHANGE_TARGET} -Dsonar.pullrequest.key=${CHANGE_ID} 
-Dsonar.token=${SONAR_TOKEN}'
           } else {
-            sh './gradlew --no-parallel --no-daemon jacocoAggregateTestReport 
sonar -PenableJacoco -Porg.sonarqube.version=4.4.1.3373 
-Dsonar.branch.name=${BRANCH_NAME} -Dsonar.login=${SONAR_TOKEN}'
+            sh './gradlew build --no-parallel --no-daemon 
jacocoAggregateTestReport sonar -PenableJacoco 
-Porg.sonarqube.version=4.4.1.3373 -Dsonar.branch.name=${BRANCH_NAME} 
-Dsonar.token=${SONAR_TOKEN}'
           }
         }
       }
diff --git a/gradle.properties b/gradle.properties
index e044c79208..ca8ca2d824 100644
--- a/gradle.properties
+++ b/gradle.properties
@@ -14,7 +14,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-org.gradle.jvmargs=-XX:+UseG1GC -Xmx1g -XX:MaxMetaspaceSize=512m
+org.gradle.jvmargs=-XX:+UseG1GC -Xmx1536m -XX:MaxMetaspaceSize=512m
 org.gradle.parallel=true
 # Build cache can be disabled with --no-build-cache option
 org.gradle.caching=true
@@ -174,6 +174,7 @@ xalan.version=2.7.1
 xercesImpl.version=2.9.1
 
 # sonar properties
+systemProp.sonar.gradle.skipCompile=true
+systemProp.sonar.host.url=https://sonarcloud.io
 systemProp.sonar.organization=apache
 systemProp.sonar.projectKey=apache_calcite
-systemProp.sonar.host.url=https://sonarcloud.io