Skip to content

Commit f5fe17c

Browse files
authored
Upgrade to Java 23 (#598)
- [x] upgrades to java 23 - [x] fixes some new "dangling docstring" warnings - [x] fixes an immutables dependency configuration issues preventing class generation Closes #561 Closes #545 Closes #589
1 parent 65ec272 commit f5fe17c

File tree

76 files changed

+1138
-1139
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

76 files changed

+1138
-1139
lines changed

.github/workflows/maven.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ on:
3333

3434
env:
3535
POM_VERSION: 2023-SNAPSHOT
36-
JAVA_VERSION: 21
36+
JAVA_VERSION: 23
3737
ERRORS_THRESHOLD: 0.01
3838
PYTHON_VERSION: "3.10"
3939

docker/benchbase/devcontainer/Dockerfile

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# For the devcontainer we need a full JDK.
2-
FROM maven:3-eclipse-temurin-21 AS devcontainer
2+
FROM maven:3-eclipse-temurin-23 AS devcontainer
33

44
LABEL org.opencontainers.image.source = "https://github.com/cmu-db/benchbase/"
55

docker/benchbase/fullimage/Dockerfile

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# TODO: Use a multi-stage build to build the fullimage from the devcontainer.
22

33
# Use a smaller base image that only has the jre, not the full jdk.
4-
FROM eclipse-temurin:21-jre AS fullimage
4+
FROM eclipse-temurin:23-jre AS fullimage
55

66
LABEL org.opencontainers.image.source = "https://github.com/cmu-db/benchbase/"
77

pom.xml

+10-10
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,9 @@
1212

1313
<properties>
1414
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
15-
<java.version>21</java.version>
16-
<maven.compiler.source>21</maven.compiler.source>
17-
<maven.compiler.target>21</maven.compiler.target>
15+
<java.version>23</java.version>
16+
<maven.compiler.source>23</maven.compiler.source>
17+
<maven.compiler.target>23</maven.compiler.target>
1818
<buildDirectory>${project.basedir}/target</buildDirectory>
1919
<!--
2020
Provids a way to limit which assembly package formats we produce.
@@ -330,13 +330,6 @@
330330
<scope>test</scope>
331331
</dependency>
332332

333-
<dependency>
334-
<groupId>org.immutables</groupId>
335-
<artifactId>value</artifactId>
336-
<version>2.10.1</version>
337-
<scope>provided</scope>
338-
</dependency>
339-
340333
<dependency>
341334
<groupId>org.codehaus.janino</groupId>
342335
<artifactId>commons-compiler</artifactId>
@@ -396,6 +389,13 @@
396389
<!-- Turn all warnings into errors to help keep the code clean. -->
397390
<arg>-Werror</arg>
398391
</compilerArgs>
392+
<annotationProcessorPaths>
393+
<path>
394+
<groupId>org.immutables</groupId>
395+
<artifactId>value</artifactId>
396+
<version>2.10.1</version>
397+
</path>
398+
</annotationProcessorPaths>
399399
</configuration>
400400
</plugin>
401401
<!-- TODO: Use PMD to check for various code health things.plugin>

src/main/java/com/oltpbenchmark/benchmarks/hyadapt/HYADAPTWorker.java

+6-6
Original file line numberDiff line numberDiff line change
@@ -119,9 +119,9 @@ protected TransactionStatus executeWork(Connection conn, TransactionType nextTra
119119
return (TransactionStatus.SUCCESS);
120120
}
121121

122-
/////////////////////////
122+
//
123123
// READ
124-
/////////////////////////
124+
//
125125

126126
private void readRecord1(Connection conn) throws SQLException {
127127
ReadRecord1 proc = this.getProcedure(ReadRecord1.class);
@@ -183,9 +183,9 @@ private void readRecord10(Connection conn) throws SQLException {
183183
proc.run(conn, key_lower_bound, new HashMap<>());
184184
}
185185

186-
/////////////////////////
186+
//
187187
// MAX
188-
/////////////////////////
188+
//
189189

190190
private void maxRecord1(Connection conn) throws SQLException {
191191
MaxRecord1 proc = this.getProcedure(MaxRecord1.class);
@@ -247,9 +247,9 @@ private void maxRecord10(Connection conn) throws SQLException {
247247
proc.run(conn, key_lower_bound);
248248
}
249249

250-
/////////////////////////
250+
//
251251
// SUM
252-
/////////////////////////
252+
//
253253

254254
private void sumRecord1(Connection conn) throws SQLException {
255255
SumRecord1 proc = this.getProcedure(SumRecord1.class);

src/main/java/com/oltpbenchmark/benchmarks/smallbank/SmallBankConstants.java

+25-25
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,28 @@
1-
/***************************************************************************
2-
* Copyright (C) 2013 by H-Store Project *
3-
* Brown University *
4-
* Massachusetts Institute of Technology *
5-
* Yale University *
6-
* *
7-
* Permission is hereby granted, free of charge, to any person obtaining *
8-
* a copy of this software and associated documentation files (the *
9-
* "Software"), to deal in the Software without restriction, including *
10-
* without limitation the rights to use, copy, modify, merge, publish, *
11-
* distribute, sublicense, and/or sell copies of the Software, and to *
12-
* permit persons to whom the Software is furnished to do so, subject to *
13-
* the following conditions: *
14-
* *
15-
* The above copyright notice and this permission notice shall be *
16-
* included in all copies or substantial portions of the Software. *
17-
* *
18-
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, *
19-
* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF *
20-
* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. *
21-
* IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR *
22-
* OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, *
23-
* ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR *
24-
* OTHER DEALINGS IN THE SOFTWARE. *
25-
***************************************************************************/
1+
/*
2+
* Copyright (C) 2013 by H-Store Project
3+
* Brown University
4+
* Massachusetts Institute of Technology
5+
* Yale University
6+
*
7+
* Permission is hereby granted, free of charge, to any person obtaining
8+
* a copy of this software and associated documentation files (the
9+
* "Software"), to deal in the Software without restriction, including
10+
* without limitation the rights to use, copy, modify, merge, publish,
11+
* distribute, sublicense, and/or sell copies of the Software, and to
12+
* permit persons to whom the Software is furnished to do so, subject to
13+
* the following conditions:
14+
*
15+
* The above copyright notice and this permission notice shall be
16+
* included in all copies or substantial portions of the Software.
17+
*
18+
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
19+
* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
20+
* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
21+
* IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR
22+
* OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
23+
* ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
24+
* OTHER DEALINGS IN THE SOFTWARE.
25+
*/
2626

2727
package com.oltpbenchmark.benchmarks.smallbank;
2828

src/main/java/com/oltpbenchmark/benchmarks/smallbank/procedures/Amalgamate.java

+25-25
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,28 @@
1-
/***************************************************************************
2-
* Copyright (C) 2013 by H-Store Project *
3-
* Brown University *
4-
* Massachusetts Institute of Technology *
5-
* Yale University *
6-
* *
7-
* Permission is hereby granted, free of charge, to any person obtaining *
8-
* a copy of this software and associated documentation files (the *
9-
* "Software"), to deal in the Software without restriction, including *
10-
* without limitation the rights to use, copy, modify, merge, publish, *
11-
* distribute, sublicense, and/or sell copies of the Software, and to *
12-
* permit persons to whom the Software is furnished to do so, subject to *
13-
* the following conditions: *
14-
* *
15-
* The above copyright notice and this permission notice shall be *
16-
* included in all copies or substantial portions of the Software. *
17-
* *
18-
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, *
19-
* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF *
20-
* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. *
21-
* IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR *
22-
* OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, *
23-
* ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR *
24-
* OTHER DEALINGS IN THE SOFTWARE. *
25-
***************************************************************************/
1+
/*
2+
* Copyright (C) 2013 by H-Store Project
3+
* Brown University
4+
* Massachusetts Institute of Technology
5+
* Yale University
6+
*
7+
* Permission is hereby granted, free of charge, to any person obtaining
8+
* a copy of this software and associated documentation files (the
9+
* "Software"), to deal in the Software without restriction, including
10+
* without limitation the rights to use, copy, modify, merge, publish,
11+
* distribute, sublicense, and/or sell copies of the Software, and to
12+
* permit persons to whom the Software is furnished to do so, subject to
13+
* the following conditions:
14+
*
15+
* The above copyright notice and this permission notice shall be
16+
* included in all copies or substantial portions of the Software.
17+
*
18+
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
19+
* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
20+
* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
21+
* IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR
22+
* OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
23+
* ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
24+
* OTHER DEALINGS IN THE SOFTWARE.
25+
*/
2626
package com.oltpbenchmark.benchmarks.smallbank.procedures;
2727

2828
import com.oltpbenchmark.api.Procedure;

src/main/java/com/oltpbenchmark/benchmarks/smallbank/procedures/Balance.java

+25-25
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,28 @@
1-
/***************************************************************************
2-
* Copyright (C) 2013 by H-Store Project *
3-
* Brown University *
4-
* Massachusetts Institute of Technology *
5-
* Yale University *
6-
* *
7-
* Permission is hereby granted, free of charge, to any person obtaining *
8-
* a copy of this software and associated documentation files (the *
9-
* "Software"), to deal in the Software without restriction, including *
10-
* without limitation the rights to use, copy, modify, merge, publish, *
11-
* distribute, sublicense, and/or sell copies of the Software, and to *
12-
* permit persons to whom the Software is furnished to do so, subject to *
13-
* the following conditions: *
14-
* *
15-
* The above copyright notice and this permission notice shall be *
16-
* included in all copies or substantial portions of the Software. *
17-
* *
18-
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, *
19-
* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF *
20-
* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. *
21-
* IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR *
22-
* OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, *
23-
* ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR *
24-
* OTHER DEALINGS IN THE SOFTWARE. *
25-
***************************************************************************/
1+
/*
2+
* Copyright (C) 2013 by H-Store Project
3+
* Brown University
4+
* Massachusetts Institute of Technology
5+
* Yale University
6+
*
7+
* Permission is hereby granted, free of charge, to any person obtaining
8+
* a copy of this software and associated documentation files (the
9+
* "Software"), to deal in the Software without restriction, including
10+
* without limitation the rights to use, copy, modify, merge, publish,
11+
* distribute, sublicense, and/or sell copies of the Software, and to
12+
* permit persons to whom the Software is furnished to do so, subject to
13+
* the following conditions:
14+
*
15+
* The above copyright notice and this permission notice shall be
16+
* included in all copies or substantial portions of the Software.
17+
*
18+
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
19+
* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
20+
* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
21+
* IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR
22+
* OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
23+
* ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
24+
* OTHER DEALINGS IN THE SOFTWARE.
25+
*/
2626
package com.oltpbenchmark.benchmarks.smallbank.procedures;
2727

2828
import com.oltpbenchmark.api.Procedure;

src/main/java/com/oltpbenchmark/benchmarks/smallbank/procedures/DepositChecking.java

+25-25
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,28 @@
1-
/***************************************************************************
2-
* Copyright (C) 2013 by H-Store Project *
3-
* Brown University *
4-
* Massachusetts Institute of Technology *
5-
* Yale University *
6-
* *
7-
* Permission is hereby granted, free of charge, to any person obtaining *
8-
* a copy of this software and associated documentation files (the *
9-
* "Software"), to deal in the Software without restriction, including *
10-
* without limitation the rights to use, copy, modify, merge, publish, *
11-
* distribute, sublicense, and/or sell copies of the Software, and to *
12-
* permit persons to whom the Software is furnished to do so, subject to *
13-
* the following conditions: *
14-
* *
15-
* The above copyright notice and this permission notice shall be *
16-
* included in all copies or substantial portions of the Software. *
17-
* *
18-
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, *
19-
* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF *
20-
* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. *
21-
* IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR *
22-
* OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, *
23-
* ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR *
24-
* OTHER DEALINGS IN THE SOFTWARE. *
25-
***************************************************************************/
1+
/*
2+
* Copyright (C) 2013 by H-Store Project
3+
* Brown University
4+
* Massachusetts Institute of Technology
5+
* Yale University
6+
*
7+
* Permission is hereby granted, free of charge, to any person obtaining
8+
* a copy of this software and associated documentation files (the
9+
* "Software"), to deal in the Software without restriction, including
10+
* without limitation the rights to use, copy, modify, merge, publish,
11+
* distribute, sublicense, and/or sell copies of the Software, and to
12+
* permit persons to whom the Software is furnished to do so, subject to
13+
* the following conditions:
14+
*
15+
* The above copyright notice and this permission notice shall be
16+
* included in all copies or substantial portions of the Software.
17+
*
18+
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
19+
* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
20+
* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
21+
* IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR
22+
* OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
23+
* ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
24+
* OTHER DEALINGS IN THE SOFTWARE.
25+
*/
2626
package com.oltpbenchmark.benchmarks.smallbank.procedures;
2727

2828
import com.oltpbenchmark.api.Procedure;

src/main/java/com/oltpbenchmark/benchmarks/smallbank/procedures/SendPayment.java

+25-25
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,28 @@
1-
/***************************************************************************
2-
* Copyright (C) 2013 by H-Store Project *
3-
* Brown University *
4-
* Massachusetts Institute of Technology *
5-
* Yale University *
6-
* *
7-
* Permission is hereby granted, free of charge, to any person obtaining *
8-
* a copy of this software and associated documentation files (the *
9-
* "Software"), to deal in the Software without restriction, including *
10-
* without limitation the rights to use, copy, modify, merge, publish, *
11-
* distribute, sublicense, and/or sell copies of the Software, and to *
12-
* permit persons to whom the Software is furnished to do so, subject to *
13-
* the following conditions: *
14-
* *
15-
* The above copyright notice and this permission notice shall be *
16-
* included in all copies or substantial portions of the Software. *
17-
* *
18-
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, *
19-
* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF *
20-
* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. *
21-
* IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR *
22-
* OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, *
23-
* ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR *
24-
* OTHER DEALINGS IN THE SOFTWARE. *
25-
***************************************************************************/
1+
/*
2+
* Copyright (C) 2013 by H-Store Project
3+
* Brown University
4+
* Massachusetts Institute of Technology
5+
* Yale University
6+
*
7+
* Permission is hereby granted, free of charge, to any person obtaining
8+
* a copy of this software and associated documentation files (the
9+
* "Software"), to deal in the Software without restriction, including
10+
* without limitation the rights to use, copy, modify, merge, publish,
11+
* distribute, sublicense, and/or sell copies of the Software, and to
12+
* permit persons to whom the Software is furnished to do so, subject to
13+
* the following conditions:
14+
*
15+
* The above copyright notice and this permission notice shall be
16+
* included in all copies or substantial portions of the Software.
17+
*
18+
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
19+
* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
20+
* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
21+
* IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR
22+
* OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
23+
* ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
24+
* OTHER DEALINGS IN THE SOFTWARE.
25+
*/
2626
package com.oltpbenchmark.benchmarks.smallbank.procedures;
2727

2828
import com.oltpbenchmark.api.Procedure;

0 commit comments

Comments
 (0)