Load world shops asynchronously and filter database queries

Author: iLemon

.gitignore

@@ -111,3 +111,6 @@ target/
 .gradle/
 !gradle/wrapper/gradle-wrapper.jar
 /build-logic
+
+# Generated Maven Shade dependency metadata
+dependency-reduced-pom.xml

quickshop-bukkit/src/main/java/com/ghostchu/quickshop/database/SimpleDatabaseHelperV2.java

@@ -533,8 +533,10 @@ public void insertTransactionRecord(@Nullable UUID from, @Nullable UUID to, fina
                        + " INNER JOIN " + DataTables.SHOP_MAP.getName()
                        + " ON " + DataTables.SHOP_MAP.getName() + ".shop = " + DataTables.SHOPS.getName() + ".id"
                        + " LEFT JOIN " + DataTables.EXTERNAL_CACHE.getName()
-                       + " ON " + DataTables.EXTERNAL_CACHE.getName() + ".shop = " + DataTables.SHOPS.getName() + ".id";
-    try(final SQLQuery query = manager.createQuery().withPreparedSQL(SQL).execute()) {
+                       + " ON " + DataTables.EXTERNAL_CACHE.getName() + ".shop = " + DataTables.SHOPS.getName() + ".id"
+                       + (worldFilter == null ? "" : " WHERE " + DataTables.SHOP_MAP.getName() + ".world = ?");
+    try(final SQLQuery query = manager.createQuery().withPreparedSQL(SQL)
+            .setParams(worldFilter == null ? java.util.List.of() : java.util.List.of(worldFilter)).execute()) {
       final ResultSet rs = query.getResultSet();
       while(rs.next()) {
         final String world = rs.getString("world");

quickshop-bukkit/src/main/java/com/ghostchu/quickshop/listener/WorldListener.java

@@ -2,6 +2,7 @@
 
 import com.ghostchu.quickshop.QuickShop;
 import com.ghostchu.quickshop.api.shop.Shop;
+import com.ghostchu.quickshop.util.Util;
 import com.ghostchu.quickshop.api.shop.ShopChunk;
 import com.ghostchu.simplereloadlib.ReloadResult;
 import com.ghostchu.simplereloadlib.ReloadStatus;
@@ -34,7 +35,21 @@ public void onWorldLoad(final WorldLoadEvent e) {
      */
     final World world = e.getWorld();
 
-    plugin.getShopLoader().loadShops(world.getName());
+    restoreWorldReferences(world);
+    // Database fetching and waiting for loader workers must not block the global tick thread.
+    Util.asyncThreadRun(() -> {
+      if(!plugin.getJavaPlugin().isEnabled() || org.bukkit.Bukkit.getWorld(world.getUID()) != world) {
+        return;
+      }
+      try {
+        plugin.getShopLoader().loadShops(world.getName());
+      } catch(final Exception exception) {
+        plugin.logger().error("Failed to load shops for world {}", world.getName(), exception);
+      }
+    });
+  }
+
+  private void restoreWorldReferences(final World world) {
     // New world data
     final Map<ShopChunk, Map<Location, Shop>> inWorld = new ConcurrentHashMap<>(1);
     // Old world data