fix(medusa): Use query relation load strategy on Products (medusajs#3926

) * fix(medusa): products retrieve uses query strategy for performance * chore: packaged version of medusa typeorm fix * chore: update path * chore: stock locations uses typeorm version * chore: resolve typeorm to medusa package * chore: Update medusa project in CI action * chore: temp disable transaction timeout for plugins --------- Co-authored-by: Oliver Windall Juhl <[email protected]> Co-authored-by: olivermrbl <[email protected]>
lrocher91 · May 2, 2023 · 4e8045a · 4e8045a
1 parent c04cbba
commit 4e8045a
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 3 deletions.
diff --git a/.changeset/rare-tips-raise.md b/.changeset/rare-tips-raise.md
@@ -0,0 +1,5 @@
+---
+"@medusajs/medusa": patch
+---
+
+fix(medusa): products retrieve uses query strategy for performance
diff --git a/integration-tests/plugins/__tests__/inventory/order/order.js b/integration-tests/plugins/__tests__/inventory/order/order.js
@@ -18,7 +18,7 @@ const {
   simpleShippingOptionFactory,
 } = require("../../../factories")
 
-jest.setTimeout(50000)
+jest.setTimeout(150000)
 
 const adminHeaders = { headers: { Authorization: "Bearer test_token" } }
 

diff --git a/integration-tests/plugins/__tests__/medusa-plugin-sendgrid/index.js b/integration-tests/plugins/__tests__/medusa-plugin-sendgrid/index.js
@@ -6,7 +6,7 @@ const { setPort, useApi } = require("../../../helpers/use-api")
 
 const adminSeeder = require("../../helpers/admin-seeder")
 
-jest.setTimeout(50000)
+jest.setTimeout(10000)
 
 const {
   simpleOrderFactory,

diff --git a/integration-tests/plugins/medusa-config.js b/integration-tests/plugins/medusa-config.js
@@ -27,6 +27,7 @@ module.exports = {
     database_type: "postgres",
     jwt_secret: "test",
     cookie_secret: "test",
+    database_extra: { idle_in_transaction_session_timeout: 0 }
   },
   modules: {
     stockLocationService: {

diff --git a/packages/medusa/src/repositories/product.ts b/packages/medusa/src/repositories/product.ts
@@ -72,7 +72,10 @@ export const ProductRepository = dataSource.getRepository(Product).extend({
   async findOne(
     options: ExtendedFindConfig<Product & ProductFilterOptions>
   ): Promise<Product | null> {
-    const queryBuilder = await this.prepareQueryBuilder_(options)
+    const options_ = { ...options }
+    options_.relationLoadStrategy = "query"
+
+    const queryBuilder = await this.prepareQueryBuilder_(options_)
     return await queryBuilder.getOne()
   },