Skip to content

Commit ee548da

Browse files
committed
Revert "Downgrade to graphql 22"
This reverts commit 8208742. # Conflicts: # graphql-builder/pom.xml
1 parent 8208742 commit ee548da

File tree

4 files changed

+21
-23
lines changed

4 files changed

+21
-23
lines changed

graphql-builder/pom.xml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
<junit.jupiter.version>5.13.0</junit.jupiter.version>
3030
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
3131
<jackson.version>2.19.0</jackson.version>
32-
<graphql.version>22.0</graphql.version>
32+
<graphql.version>24.1</graphql.version>
3333
</properties>
3434

3535
<licenses>
@@ -49,12 +49,12 @@
4949
<dependency>
5050
<groupId>com.graphql-java</groupId>
5151
<artifactId>graphql-java-extended-scalars</artifactId>
52-
<version>${graphql.version}</version>
52+
<version>22.0</version>
5353
</dependency>
5454
<dependency>
5555
<groupId>com.graphql-java</groupId>
5656
<artifactId>graphql-java-extended-validation</artifactId>
57-
<version>${graphql.version}</version>
57+
<version>22.0</version>
5858
</dependency>
5959
<dependency>
6060
<groupId>org.reflections</groupId>

graphql-builder/src/test/java/com/phocassoftware/graphql/builder/DirectiveTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ public void testDirectiveArgumentDefinition() {
8989
List<LinkedHashMap<String, Object>> dir = (List<LinkedHashMap<String, Object>>) ((Map<String, Object>) response.get("__schema")).get("directives");
9090
LinkedHashMap<String, Object> input = dir.stream().filter(map -> map.get("name").equals("Input")).collect(Collectors.toList()).get(0);
9191

92-
assertEquals(30, dir.size());
92+
assertEquals(32, dir.size());
9393
assertEquals("ARGUMENT_DEFINITION", ((List<String>) input.get("locations")).get(0));
9494
assertEquals(1, ((List<Object>) input.get("args")).size());
9595
// getNickname(nickName: String! @Input(value : "TT")): String!

graphql-builder/src/test/java/com/phocassoftware/graphql/builder/JakartaValidationDirectiveTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ void testJakartaSizeDirectiveArgumentDefinition() {
5353
List<LinkedHashMap<String, Object>> dir = (List<LinkedHashMap<String, Object>>) ((Map<String, Object>) response.get("__schema")).get("directives");
5454
LinkedHashMap<String, Object> constraint = dir.stream().filter(map -> map.get("name").equals("Size")).collect(Collectors.toList()).get(0);
5555

56-
assertEquals(30, dir.size());
56+
assertEquals(32, dir.size());
5757
assertEquals("ARGUMENT_DEFINITION", ((List<String>) constraint.get("locations")).get(0));
5858
assertEquals("INPUT_FIELD_DEFINITION", ((List<String>) constraint.get("locations")).get(1));
5959
assertEquals(5, ((List<Object>) constraint.get("args")).size());

graphql-database-manager-core/src/main/java/com/phocassoftware/graphql/database/manager/Database.java

Lines changed: 16 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@
3535
import java.util.stream.Stream;
3636

3737
import org.dataloader.DataLoader;
38+
import org.dataloader.DataLoaderFactory;
3839
import org.dataloader.DataLoaderOptions;
3940

4041
@SuppressWarnings("unchecked")
@@ -61,32 +62,29 @@ public class Database {
6162
this.submitted = new AtomicInteger();
6263

6364
items = new TableDataLoader<>(
64-
new DataLoader<DatabaseKey<Table>, Table>(
65-
keys -> {
66-
return driver.get(keys);
67-
},
68-
DataLoaderOptions.newOptions().setMaxBatchSize(driver.maxBatchSize())
69-
),
65+
DataLoaderFactory
66+
.newDataLoader(
67+
driver::get,
68+
DataLoaderOptions.newOptions().setMaxBatchSize(driver.maxBatchSize()).build()
69+
),
7070
this::handleFuture
7171
); // will auto call global
7272

7373
queries = new TableDataLoader<>(
74-
new DataLoader<DatabaseQueryKey<Table>, List<Table>>(
75-
keys -> {
76-
return merge(keys.stream().map(driver::query));
77-
},
78-
DataLoaderOptions.newOptions().setBatchingEnabled(false)
79-
),
74+
DataLoaderFactory
75+
.newDataLoader(
76+
keys -> merge(keys.stream().map(driver::query)),
77+
DataLoaderOptions.newOptions().setBatchingEnabled(false).build()
78+
),
8079
this::handleFuture
8180
); // will auto call global
8281

8382
queryHistories = new TableDataLoader<>(
84-
new DataLoader<DatabaseQueryHistoryKey<Table>, List<Table>>(
85-
keys -> {
86-
return merge(keys.stream().map(driver::queryHistory));
87-
},
88-
DataLoaderOptions.newOptions().setBatchingEnabled(false)
89-
),
83+
DataLoaderFactory
84+
.newDataLoader(
85+
keys -> merge(keys.stream().map(driver::queryHistory)),
86+
DataLoaderOptions.newOptions().setBatchingEnabled(false).build()
87+
),
9088
this::handleFuture
9189
); // will auto call global
9290

0 commit comments

Comments
 (0)