MDEE-596: Missed index causes FILE SORT and query slowness (#350)

* MDEE-596: Missed index causes FILE SORT and query slowness
mage-os · Oct 31, 2023 · 00e1a84 · 00e1a84
1 parent d630c28
commit 00e1a84
Show file tree

Hide file tree

Showing 5 changed files with 37 additions and 2 deletions.
diff --git a/CatalogDataExporter/etc/db_schema.xml b/CatalogDataExporter/etc/db_schema.xml
@@ -77,6 +77,10 @@
             <column name="modified_at"/>
             <column name="id"/>
         </index>
+        <index referenceId="catalog_data_exporter_products_modified_at_status" indexType="btree">
+            <column name="modified_at"/>
+            <column name="status"/>
+        </index>
     </table>
 
     <table name="catalog_data_exporter_categories" resource="default" engine="innodb" comment="Category Feed Storage">

diff --git a/CatalogDataExporter/etc/db_schema_whitelist.json b/CatalogDataExporter/etc/db_schema_whitelist.json
@@ -1,7 +1,8 @@
 {
     "catalog_data_exporter_products": {
         "index": {
-            "CATALOG_DATA_EXPORTER_PRODUCTS_MODIFIED_AT": true
+            "CATALOG_DATA_EXPORTER_PRODUCTS_MODIFIED_AT": true,
+            "CATALOG_DATA_EXPORTER_PRODUCTS_MODIFIED_AT_STATUS": true
         },
         "constraint": {
             "PRIMARY": true

diff --git a/ProductPriceDataExporter/etc/db_schema.xml b/ProductPriceDataExporter/etc/db_schema.xml
@@ -78,5 +78,9 @@
             <column name="modified_at"/>
             <column name="product_id"/>
         </index>
+        <index referenceId="catalog_data_exporter_product_prices_modified_at_status" indexType="btree">
+            <column name="modified_at"/>
+            <column name="status"/>
+        </index>
     </table>
 </schema>
diff --git a/ProductPriceDataExporter/etc/db_schema_whitelist.json b/ProductPriceDataExporter/etc/db_schema_whitelist.json
@@ -1,7 +1,8 @@
 {
     "catalog_data_exporter_product_prices": {
         "index": {
-            "CATALOG_DATA_EXPORTER_PRODUCT_PRICES_MODIFIED_AT": true
+            "CATALOG_DATA_EXPORTER_PRODUCT_PRICES_MODIFIED_AT": true,
+            "CATALOG_DATA_EXPORTER_PRODUCT_PRICES_MODIFIED_AT_STATUS": true
         },
         "constraint": {
             "PRIMARY": true

diff --git a/.../testsuite/Magento/Setup/Declaration/_files/ignore_whitelisting/commerce-data-export.json b/.../testsuite/Magento/Setup/Declaration/_files/ignore_whitelisting/commerce-data-export.json
@@ -0,0 +1,25 @@
+{
+  "catalog_data_exporter_products": {
+    "constraint": [
+      "PRIMARY"
+    ],
+    "index": [
+      "CATALOG_DATA_EXPORTER_PRODUCTS_MODIFIED_AT_ID",
+      "CATALOG_DATA_EXPORTER_PRODUCTS_MODIFIED_AT_STATUS"
+    ]
+  },
+  "catalog_data_exporter_product_prices": {
+    "constraint": [
+      "PRIMARY"
+    ],
+    "index": [
+      "CATALOG_DATA_EXPORTER_PRODUCT_PRICES_MODIFIED_AT_PRODUCT_ID",
+      "CATALOG_DATA_EXPORTER_PRODUCT_PRICES_MODIFIED_AT_STATUS"
+    ]
+  },
+  "catalog_data_submitted_hash": {
+    "constraint": [
+      "PRIMARY"
+    ]
+  }
+}