Browse Source

[Quarkus] Replace lambdas with method references and remove unnecessary casts (#8262)

* [Quarkus] Replace lambdas with method references

Also, remove some unnecessary casts.

* Updating to retrigger github actions

---------

Co-authored-by: Nate <[email protected]>
Ziad Hatahet 2 years ago
parent
commit
817e2cc9d6

+ 2 - 2
frameworks/Java/quarkus/resteasy-reactive-hibernate-reactive/src/main/java/io/quarkus/benchmark/repository/WorldRepository.java

@@ -56,7 +56,7 @@ public class WorldRepository extends BaseRepository {
         final List<World> worlds = new ArrayList<>(count);
         Uni<Void> loopRoot = Uni.createFrom().voidItem();
         for (int i = 0; i < count; i++) {
-            loopRoot = loopRoot.chain(() -> s.get(World.class, localRandom.getNextRandom()).invoke(word -> worlds.add(word)).replaceWithVoid());
+            loopRoot = loopRoot.chain(() -> s.get(World.class, localRandom.getNextRandom()).invoke(worlds::add).replaceWithVoid());
         }
         return loopRoot.map(v -> worlds);
     }
@@ -70,7 +70,7 @@ public class WorldRepository extends BaseRepository {
         final LocalRandom localRandom = Randomizer.current();
         Uni<Void> loopRoot = Uni.createFrom().voidItem();
         for (int i = 0; i < count; i++) {
-            loopRoot = loopRoot.chain(() -> s.find(World.class, localRandom.getNextRandom()).invoke(word -> worlds.add(word)).replaceWithVoid());
+            loopRoot = loopRoot.chain(() -> s.find(World.class, localRandom.getNextRandom()).invoke(worlds::add).replaceWithVoid());
         }
         return loopRoot.map(v -> worlds);
     }

+ 1 - 1
frameworks/Java/quarkus/resteasy-reactive-hibernate-reactive/src/main/java/io/quarkus/benchmark/resource/FortuneResource.java

@@ -34,7 +34,7 @@ public class FortuneResource  {
     @Inject
     FortuneRepository repository;
 
-    private static final Comparator<Fortune> fortuneComparator = Comparator.comparing(fortune -> fortune.getMessage());
+    private static final Comparator<Fortune> fortuneComparator = Comparator.comparing(Fortune::getMessage);
 
     @Produces("text/html; charset=UTF-8")
     @GET

+ 3 - 3
frameworks/Java/quarkus/resteasy-reactive-hibernate/src/main/java/io/quarkus/benchmark/repository/WorldRepository.java

@@ -39,7 +39,7 @@ public class WorldRepository {
 
     public World loadSingleWorldById(Integer id) {
         try (StatelessSession ss = sf.openStatelessSession()) {
-            return (World) ss.get(World.class, id);
+            return ss.get(World.class, id);
         }
     }
 
@@ -49,7 +49,7 @@ public class WorldRepository {
         try (StatelessSession ss = sf.openStatelessSession()) {
             //The rules require individual load: we can't use the Hibernate feature which allows load by multiple IDs as one single operation
             for (int i=0;i<count;i++) {
-                list[i] = (World) ss.get(World.class, random.getNextRandom());
+                list[i] = ss.get(World.class, random.getNextRandom());
             }
             return list;
         }
@@ -67,7 +67,7 @@ public class WorldRepository {
                 // # vi. At least the randomNumber field must be read from the database result set.
                 final int previousRead = w.getRandomNumber();
                 //Update it, but make sure to exclude the current number as Hibernate optimisations would otherwise
-                //skip the write operation:
+                // skip the write operation:
                 w.setRandomNumber(random.getNextRandomExcluding(previousRead));
                 s.update(w);
             }

+ 1 - 1
frameworks/Java/quarkus/resteasy-reactive-hibernate/src/main/java/io/quarkus/benchmark/resource/FortuneResource.java

@@ -40,7 +40,7 @@ public class FortuneResource {
     @Inject
     FortuneRepository repository;
 
-    private static final Comparator<Fortune> fortuneComparator = Comparator.comparing(fortune -> fortune.getMessage());
+    private static final Comparator<Fortune> fortuneComparator = Comparator.comparing(Fortune::getMessage);
 
     @GET
     @Path("/fortunes")