Как оптимизировать запрос Mysql?

Question

[Хуршед Абдужалил]

Есть запросы где делаются запросы на подобии:

SELECT count(distinct(product.id))
FROM `product`
JOIN `product_to_category` `p2c` ON product.id = p2c.product_id
JOIN `city` `c` ON c.id = product.city_id
LEFT JOIN `city` `city` ON city.id=product.city_id
LEFT JOIN `company` ON company.id=product.company_id
WHERE ((`product`.`publish` = 1)) AND (c.id = 108 or c.parent_id = 108) AND ((`p2c`.`category_id` IN (917, 918, 919, 920, 921, 922, 923, 924, 925, 926, 927, 928, 929, 930, 931, 932, 933, 934, 935, 936, 937, 938, 939, 940, 941, 942, 943, 944, 945, 946, 947, 948, 949, 950)));

я как понимаю в основном тут вот эта часть долго выполняется при нагрузке

((`p2c`.`category_id` IN (917, 918, 919, 920, 921, 922, 923, 924, 925, 926, 927, 928, 929, 930, 931, 932, 933, 934, 935, 936, 937, 938, 939, 940, 941, 942, 943, 944, 945, 946, 947, 948, 949, 950))

как можно оптимизировать?

Answer 1

[Илья]

Сделайте EXPLAIN запроса и потом уже смотрите что не так.
И на первый взгляд DISTINCT лишний, т.к. ID должен быть уникальным (если нет, то по первичному ключу получайте количество).

---

Вот этот JOIN лишний:
LEFT JOIN `city` `city` ON city.id=product.city_id

Comments

[Хуршед Абдужалил]

И на первый взгляд DISTINCT лишний, т.к. ID должен быть уникальным (если нет, то по первичному ключу получайте количество).

из за того что в таблицу product_to_category может быть несколько записей одного продукта product_id то без distinct получаются дубли, поэтому написал distinct

Answer 2

[Rsa97]

LEFT JOIN'ы не нужны - их данные нигде не используются. Условия из WHERE можно попробовать перенести в ON.

SELECT COUNT(DISTINCT `p`.`id`)
  FROM `product` AS `p`
  JOIN `product_to_category` AS `p2c` ON `p`.`id` = `p2c`.`product_id`
    AND `p`.`publish` = 1
    AND `p2c`.`category_id` IN (917, 918, 919, 920, 921, 922, 923, 924, 925, 926, 927, 928, 929, 930, 931, 932, 933, 934, 935, 936, 937, 938, 939, 940, 941, 942, 943, 944, 945, 946, 947, 948, 949, 950)
  JOIN `city` AS `c` ON `c`.`id` = `p`.`city_id`
    AND (`c`.`id` = 108 OR `c`.`parent_id` = 108)

Ну и правильные индексы, тут уже смотрите EXPLAIN/

Comments

[Хуршед Абдужалил]

ну вот эту часть прям никак по другому не перезаписать?

AND `p2c`.`category_id` IN (917, 918, 919, 920, 921, 922, 923, 924, 925, 926, 927, 928, 929, 930, 931, 932, 933, 934, 935, 936, 937, 938, 939, 940, 941, 942, 943, 944, 945, 946, 947, 948, 949, 950)

[Rsa97]

Хуршед Абдужалил: Только если через OR, но будет ещё медленнее.

[Хуршед Абдужалил]

Rsa97: ага и это печально...

[Fortop]

Хуршед Абдужалил:
Конкретно эту?

category_id BETWEEN 916 AND 951

[Хуршед Абдужалил]

Fortop: нет, там разные сочетания могут быть

[Fortop]

Хуршед Абдужалил: откуда вы этот список берете?

[Хуршед Абдужалил]

Fortop: ну это подразделы. когда находимся в главном разделе, берем все продукты подразделов этого раздела типа так)

[Fortop]

Хуршед Абдужалил:
Ну так выполните такой же JOIN для подразделов

SELECT products.* FROM products
JOIN categories ON products.category_id = categories.id
JOIN categories as sub_category ON sub_category.parent_id = categories.id
WHERE categories.id = 1

[Rsa97]

Fortop: Второй JOIN смысла не имеет. Достаточно

SELECT `p`.* 
  FROM `products` AS `p`
  JOIN `categories` AS `c` ON `c`.`id` = `p`.`category_id`
    AND `c`.`parent_id` = 1

[Fortop]

Rsa97: Конкретно в моем псевдокоде не имеет.

В рамках структуры автора - без понятия.

Суть в том чтобы весь список айди в условии IN () выбирать через JOIN