This is an automated email from the ASF dual-hosted git repository. mhubail pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/asterixdb.git
commit 61524a67111c434fd2e0de900f79c3478c38dd83 Author: Wail Alkowaileet <[email protected]> AuthorDate: Wed May 1 15:09:24 2024 -0700 [NO ISSUE][OTH] Change mvn Java ver. to 17 - user model changes: no - storage format changes: no - interface changes: no Details: The change of parameter/variable names is to avoid 'cannot be used as an identifier as it is a keyword' being thrown by impsort-maven-plugin. Change-Id: I13ec2c1063754b93d8fce551362934d0d1c593d5 Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18262 Tested-by: Jenkins <[email protected]> Contrib: Michael Blow <[email protected]> Reviewed-by: Michael Blow <[email protected]> --- asterixdb/asterix-app/pom.xml | 1 + .../TestPrimaryIndexOperationTrackerFactory.java | 11 ----------- asterixdb/pom.xml | 1 + .../hyracks/storage/common/buffercache/IRateLimiter.java | 2 +- .../storage/common/buffercache/NoOpRateLimiter.java | 2 +- .../storage/common/buffercache/SleepRateLimiter.java | 5 ++--- .../am/lsm/btree/LSMBTreePageWriteCallbackTest.java | 4 ++-- .../hyracks/storage/am/lsm/btree/impl/TestLsmBtree.java | 16 ++++++++-------- hyracks-fullstack/pom.xml | 3 ++- 9 files changed, 18 insertions(+), 27 deletions(-) diff --git a/asterixdb/asterix-app/pom.xml b/asterixdb/asterix-app/pom.xml index ca5908a964..b7e3b946d8 100644 --- a/asterixdb/asterix-app/pom.xml +++ b/asterixdb/asterix-app/pom.xml @@ -409,6 +409,7 @@ -Xdebug -Xrunjdwp:transport=dt_socket,server=y,address=8000,suspend=${debug.suspend.flag} -Xloggc:"${project.build.directory}/surefire-reports/SqlppExecutionTest-%p-gc.log" -XX:+PrintGC -XX:+PrintGCDateStamps -XX:GCLogFileSize=10M + --add-opens=jdk.management/com.sun.management.internal=ALL-UNNAMED --add-opens=java.management/sun.management=ALL-UNNAMED --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.nio=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED </argLine> </configuration> <goals> diff --git a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/dataflow/TestPrimaryIndexOperationTrackerFactory.java b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/dataflow/TestPrimaryIndexOperationTrackerFactory.java index c4390fa52c..38fdf5689d 100644 --- a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/dataflow/TestPrimaryIndexOperationTrackerFactory.java +++ b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/dataflow/TestPrimaryIndexOperationTrackerFactory.java @@ -19,7 +19,6 @@ package org.apache.asterix.test.dataflow; import java.lang.reflect.Field; -import java.lang.reflect.Modifier; import java.util.Map; import org.apache.asterix.common.api.INcApplicationContext; @@ -70,20 +69,10 @@ public class TestPrimaryIndexOperationTrackerFactory extends PrimaryIndexOperati } } - static void setFinal(Field field, Object obj, Object newValue) throws Exception { - field.setAccessible(true); - Field modifiersField = Field.class.getDeclaredField("modifiers"); - modifiersField.setAccessible(true); - modifiersField.setInt(field, field.getModifiers() & ~Modifier.FINAL); - field.set(obj, newValue); - } - @SuppressWarnings({ "rawtypes", "unchecked" }) static void replaceMapEntry(Field field, Object obj, Object key, Object value) throws Exception, IllegalAccessException { field.setAccessible(true); - Field modifiersField = Field.class.getDeclaredField("modifiers"); - modifiersField.setAccessible(true); Map map = (Map) field.get(obj); map.put(key, value); } diff --git a/asterixdb/pom.xml b/asterixdb/pom.xml index 93a4edb953..2c241626d8 100644 --- a/asterixdb/pom.xml +++ b/asterixdb/pom.xml @@ -146,6 +146,7 @@ -DrunSlowAQLTests=${runSlowAQLTests} -Xdebug -Xrunjdwp:transport=dt_socket,server=y,address=8000,suspend=${debug.suspend.flag} + --add-opens=jdk.management/com.sun.management.internal=ALL-UNNAMED --add-opens=java.management/sun.management=ALL-UNNAMED --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.nio=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED ${coverageArgLine} ${extraSurefireArgLine} </argLine> diff --git a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/IRateLimiter.java b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/IRateLimiter.java index b7433ae2cd..163cbafc68 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/IRateLimiter.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/IRateLimiter.java @@ -24,5 +24,5 @@ public interface IRateLimiter { void setRate(double ratePerSecond); - void request(int permits) throws HyracksDataException; + void request(int numOfPermits) throws HyracksDataException; } diff --git a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/NoOpRateLimiter.java b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/NoOpRateLimiter.java index ac0a1a9d27..a0660f6300 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/NoOpRateLimiter.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/NoOpRateLimiter.java @@ -33,7 +33,7 @@ public class NoOpRateLimiter implements IRateLimiter { } @Override - public void request(int tokens) throws HyracksDataException { + public void request(int numOfPermits) throws HyracksDataException { // no op } diff --git a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/SleepRateLimiter.java b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/SleepRateLimiter.java index 4d0ca9217c..3dcbfb88c3 100644 --- a/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/SleepRateLimiter.java +++ b/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/SleepRateLimiter.java @@ -26,7 +26,6 @@ import com.google.common.util.concurrent.RateLimiter; /** * A wrapper of the RateLimiter implementation from {@link RateLimiter} - * */ public class SleepRateLimiter implements IRateLimiter { /** @@ -55,8 +54,8 @@ public class SleepRateLimiter implements IRateLimiter { } @Override - public void request(int permits) throws HyracksDataException { - rateLimiterImpl.acquire(permits); + public void request(int numOfPermits) throws HyracksDataException { + rateLimiterImpl.acquire(numOfPermits); } } diff --git a/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/LSMBTreePageWriteCallbackTest.java b/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/LSMBTreePageWriteCallbackTest.java index 4e59ab0f5e..cf3b3a9d01 100644 --- a/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/LSMBTreePageWriteCallbackTest.java +++ b/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/LSMBTreePageWriteCallbackTest.java @@ -68,8 +68,8 @@ public class LSMBTreePageWriteCallbackTest extends OrderedIndexTestDriver { } @Override - public void request(int permits) throws HyracksDataException { - limiter.request(permits); + public void request(int numOfPermits) throws HyracksDataException { + limiter.request(numOfPermits); pageCounter++; } }; diff --git a/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/impl/TestLsmBtree.java b/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/impl/TestLsmBtree.java index 2d0f079120..10bd594289 100644 --- a/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/impl/TestLsmBtree.java +++ b/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/impl/TestLsmBtree.java @@ -183,20 +183,20 @@ public class TestLsmBtree extends LSMBTree { } } - public void allowModify(int permits) { - modifySemaphore.release(permits); + public void allowModify(int numOfPermits) { + modifySemaphore.release(numOfPermits); } - public void allowSearch(int permits) { - searchSemaphore.release(permits); + public void allowSearch(int numOfPermits) { + searchSemaphore.release(numOfPermits); } - public void allowFlush(int permits) { - flushSemaphore.release(permits); + public void allowFlush(int numOfPermits) { + flushSemaphore.release(numOfPermits); } - public void allowMerge(int permits) { - mergeSemaphore.release(permits); + public void allowMerge(int numOfPermits) { + mergeSemaphore.release(numOfPermits); } @Override diff --git a/hyracks-fullstack/pom.xml b/hyracks-fullstack/pom.xml index 9fecea19d6..fce8c39fae 100644 --- a/hyracks-fullstack/pom.xml +++ b/hyracks-fullstack/pom.xml @@ -49,7 +49,7 @@ <root.dir>${basedir}</root.dir> <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding> <file.encoding>UTF-8</file.encoding> - <maven.compiler.release>11</maven.compiler.release> + <maven.compiler.release>17</maven.compiler.release> <javac.xlint.value>all</javac.xlint.value> <jvm.extraargs /> <sonar.jacoco.reportPath>${env.PWD}/target/jacoco-merged.exec</sonar.jacoco.reportPath> @@ -544,6 +544,7 @@ <reuseForks>false</reuseForks> <argLine>-enableassertions -Xmx2048m -Dfile.encoding=UTF-8 -Xdebug -Xrunjdwp:transport=dt_socket,server=y,address=8000,suspend=n ${coverageArgLine} + --add-opens=jdk.management/com.sun.management.internal=ALL-UNNAMED --add-opens=java.management/sun.management=ALL-UNNAMED --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.nio=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.base/java.io=ALL-UNNAMED </argLine> <systemPropertyVariables> <log4j.configurationFile>${testLog4jConfigFile}</log4j.configurationFile>
