Remove forbidden use of @batchsize

bukajsytlos · Brutus5000 · commit 375c618f6a29 · 2024-06-14T23:44:00.000+02:00
diff --git a/src/main/java/com/faforever/api/data/domain/Game.java b/src/main/java/com/faforever/api/data/domain/Game.java
@@ -139,7 +139,6 @@ public Set<GameReview> getReviews() {
   @OneToOne(fetch = FetchType.LAZY)
   @PrimaryKeyJoinColumn
   @UpdatePermission(expression = Prefab.ALL)
-  @BatchSize(size = 1000)
   public GameReviewsSummary getReviewsSummary() {
     return reviewsSummary;
   }
diff --git a/src/main/java/com/faforever/api/data/domain/GameReviewsSummary.java b/src/main/java/com/faforever/api/data/domain/GameReviewsSummary.java
@@ -2,7 +2,6 @@
 
 import com.yahoo.elide.annotation.Include;
 import lombok.Setter;
-import org.hibernate.annotations.BatchSize;
 import org.hibernate.annotations.Immutable;
 
 import jakarta.annotation.Nullable;
@@ -65,7 +64,6 @@ public Float getAverageScore() {
   }
 
   @OneToOne(mappedBy = "reviewsSummary")
-  @BatchSize(size = 1000)
   public Game getGame() {
     return game;
   }
diff --git a/src/main/java/com/faforever/api/data/domain/Map.java b/src/main/java/com/faforever/api/data/domain/Map.java
@@ -96,7 +96,6 @@ public List<MapVersion> getVersions() {
   @ManyToOne(fetch = FetchType.LAZY)
   @JoinColumn(name = "author")
   @Nullable
-  @BatchSize(size = 1000)
   public Player getAuthor() {
     return author;
   }
@@ -114,7 +113,6 @@ public MapStatistics getStatistics() {
           referencedColumnName = "id")
       )
   })
-  @BatchSize(size = 1000)
   public MapVersion getLatestVersion() {
     return latestVersion;
   }
@@ -134,7 +132,6 @@ public MapReviewsSummary getReviewsSummary() {
   @ManyToOne
   @JoinColumn(name = "license")
   @Nullable
-  @BatchSize(size = 1000)
   public License getLicense() {
     return license;
   }
diff --git a/src/main/java/com/faforever/api/data/domain/MapReviewsSummary.java b/src/main/java/com/faforever/api/data/domain/MapReviewsSummary.java
@@ -2,7 +2,6 @@
 
 import com.yahoo.elide.annotation.Include;
 import lombok.Setter;
-import org.hibernate.annotations.BatchSize;
 import org.hibernate.annotations.Immutable;
 
 import jakarta.annotation.Nullable;
@@ -70,7 +69,6 @@ public Float getAverageScore() {
 
   @OneToOne(fetch = FetchType.LAZY)
   @JoinColumn(name = "map_id", insertable = false, updatable = false)
-  @BatchSize(size = 1000)
   public Map getMap() {
     return map;
   }
diff --git a/src/main/java/com/faforever/api/data/domain/MapVersion.java b/src/main/java/com/faforever/api/data/domain/MapVersion.java
@@ -11,7 +11,6 @@
 import com.yahoo.elide.annotation.Include;
 import com.yahoo.elide.annotation.UpdatePermission;
 import lombok.Setter;
-import org.hibernate.annotations.BatchSize;
 
 import jakarta.persistence.Column;
 import jakarta.persistence.Entity;
@@ -112,7 +111,6 @@ public boolean isHidden() {
   @ManyToOne(fetch = FetchType.LAZY)
   @JoinColumn(name = "map_id")
   @NotNull
-  @BatchSize(size = 1000)
   public Map getMap() {
     return this.map;
   }
diff --git a/src/main/java/com/faforever/api/data/domain/MapVersionReviewsSummary.java b/src/main/java/com/faforever/api/data/domain/MapVersionReviewsSummary.java
@@ -2,7 +2,6 @@
 
 import com.yahoo.elide.annotation.Include;
 import lombok.Setter;
-import org.hibernate.annotations.BatchSize;
 import org.hibernate.annotations.Immutable;
 
 import jakarta.annotation.Nullable;
@@ -70,7 +69,6 @@ public Float getAverageScore() {
 
   @OneToOne(fetch = FetchType.LAZY)
   @JoinColumn(name = "map_version_id", insertable = false, updatable = false)
-  @BatchSize(size = 1000)
   public MapVersion getMapVersion() {
     return mapVersion;
   }
diff --git a/src/main/java/com/faforever/api/data/domain/MapVersionStatistics.java b/src/main/java/com/faforever/api/data/domain/MapVersionStatistics.java
@@ -2,7 +2,6 @@
 
 import com.yahoo.elide.annotation.Include;
 import lombok.Setter;
-import org.hibernate.annotations.BatchSize;
 import org.hibernate.annotations.Immutable;
 
 import jakarta.persistence.Column;
@@ -48,7 +47,6 @@ public Integer getDraws() {
 
   @OneToOne(fetch = FetchType.LAZY)
   @JoinColumn(name = "map_version_id", insertable = false, updatable = false)
-  @BatchSize(size = 1000)
   public MapVersion getMapVersion() {
     return mapVersion;
   }
diff --git a/src/main/java/com/faforever/api/data/domain/Mod.java b/src/main/java/com/faforever/api/data/domain/Mod.java
@@ -6,7 +6,6 @@
 import com.yahoo.elide.annotation.Include;
 import com.yahoo.elide.annotation.UpdatePermission;
 import lombok.Setter;
-import org.hibernate.annotations.BatchSize;
 import org.hibernate.annotations.JoinColumnOrFormula;
 import org.hibernate.annotations.JoinColumnsOrFormulas;
 import org.hibernate.annotations.JoinFormula;
@@ -114,7 +113,6 @@ public ModReviewsSummary getReviewsSummary() {
   @ManyToOne
   @JoinColumn(name = "license")
   @Nullable
-  @BatchSize(size = 1000)
   public License getLicense() {
     return license;
   }
diff --git a/src/main/java/com/faforever/api/data/domain/ModReviewsSummary.java b/src/main/java/com/faforever/api/data/domain/ModReviewsSummary.java
@@ -2,7 +2,6 @@
 
 import com.yahoo.elide.annotation.Include;
 import lombok.Setter;
-import org.hibernate.annotations.BatchSize;
 import org.hibernate.annotations.Immutable;
 
 import jakarta.annotation.Nullable;
@@ -70,7 +69,6 @@ public Float getAverageScore() {
 
   @OneToOne(fetch = FetchType.LAZY)
   @JoinColumn(name = "mod_id", insertable = false, updatable = false)
-  @BatchSize(size = 1000)
   public Mod getMod() {
     return mod;
   }
diff --git a/src/main/java/com/faforever/api/data/domain/ModVersionReviewsSummary.java b/src/main/java/com/faforever/api/data/domain/ModVersionReviewsSummary.java
@@ -2,7 +2,6 @@
 
 import com.yahoo.elide.annotation.Include;
 import lombok.Setter;
-import org.hibernate.annotations.BatchSize;
 import org.hibernate.annotations.Immutable;
 
 import jakarta.annotation.Nullable;
@@ -70,7 +69,6 @@ public Float getAverageScore() {
 
   @OneToOne(fetch = FetchType.LAZY)
   @JoinColumn(name = "mod_version_id", insertable = false, updatable = false)
-  @BatchSize(size = 1000)
   public ModVersion getModVersion() {
     return modVersion;
   }

Original file line number	Diff line number	Diff line change
`@@ -139,7 +139,6 @@ public Set<GameReview> getReviews() {`
`139`	`139`	`@OneToOne(fetch = FetchType.LAZY)`
`140`	`140`	`@PrimaryKeyJoinColumn`
`141`	`141`	`@UpdatePermission(expression = Prefab.ALL)`
`142`		`- @BatchSize(size = 1000)`
`143`	`142`	`public GameReviewsSummary getReviewsSummary() {`
`144`	`143`	`return reviewsSummary;`
`145`	`144`	`}`
Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,6 @@`
`2`	`2`
`3`	`3`	`import com.yahoo.elide.annotation.Include;`
`4`	`4`	`import lombok.Setter;`
`5`		`-import org.hibernate.annotations.BatchSize;`
`6`	`5`	`import org.hibernate.annotations.Immutable;`
`7`	`6`
`8`	`7`	`import jakarta.annotation.Nullable;`
`@@ -65,7 +64,6 @@ public Float getAverageScore() {`
`65`	`64`	`}`
`66`	`65`
`67`	`66`	`@OneToOne(mappedBy = "reviewsSummary")`
`68`		`- @BatchSize(size = 1000)`
`69`	`67`	`public Game getGame() {`
`70`	`68`	`return game;`
`71`	`69`	`}`
Original file line number	Diff line number	Diff line change
`@@ -96,7 +96,6 @@ public List<MapVersion> getVersions() {`
`96`	`96`	`@ManyToOne(fetch = FetchType.LAZY)`
`97`	`97`	`@JoinColumn(name = "author")`
`98`	`98`	`@Nullable`
`99`		`- @BatchSize(size = 1000)`
`100`	`99`	`public Player getAuthor() {`
`101`	`100`	`return author;`
`102`	`101`	`}`
`@@ -114,7 +113,6 @@ public MapStatistics getStatistics() {`
`114`	`113`	`referencedColumnName = "id")`
`115`	`114`	`)`
`116`	`115`	`})`
`117`		`- @BatchSize(size = 1000)`
`118`	`116`	`public MapVersion getLatestVersion() {`
`119`	`117`	`return latestVersion;`
`120`	`118`	`}`
`@@ -134,7 +132,6 @@ public MapReviewsSummary getReviewsSummary() {`
`134`	`132`	`@ManyToOne`
`135`	`133`	`@JoinColumn(name = "license")`
`136`	`134`	`@Nullable`
`137`		`- @BatchSize(size = 1000)`
`138`	`135`	`public License getLicense() {`
`139`	`136`	`return license;`
`140`	`137`	`}`