Оптимизация запроса в seo_pro.php

[maxway777]

Всем привет, в файле seo_pro.php есть запрос $query = $this->db->query("SELECT LOWER(`keyword`) as 'keyword', `query` FROM " . DB_PREFIX . "url_alias ORDER BY url_alias_id");
Запрос выполняется 1 сек, можно как то его оптимизировать? Товаров больше 50к.

Файл по пути catalog/controller/startup/seo_pro.php

Lightning указывает именно на эту строчку.

Ocstore 2.3
 

 

Змінено 9 серпня 2023 користувачем maxway777

[buslikdrev]

Попробуйте удалить сортировку  ORDER BY url_alias_id

[markimax]

В 09.08.2023 в 15:23, maxway777 сказав:

Всем привет, в файле seo_pro.php есть запрос $query = $this->db->query("SELECT LOWER(`keyword`) as 'keyword', `query` FROM " . DB_PREFIX . "url_alias ORDER BY url_alias_id");
 

 

 

 

Це самий простіший запит, який повинен виконуватися миттєво
Просто у вас сервер "підвис", виставився в чергу та спотикнувся на цьому запиті, іншим разом на іншому спотикнеться
Тут не в запиті діло.

[buslikdrev]

Также можно попробовать убрать перевод в нижний регистр через sql LOWER(`keyword`) as 'keyword' и сделать через php mb_strtolower($row['keyword'], 'UTF-8')

[maxway777]

09.08.2023 в 15:42, markimax сказал:

Це самий простіший запит, який повинен виконуватися миттєво
Просто у вас сервер "підвис", виставився в чергу та спотикнувся на цьому запиті, іншим разом на іншому спотикнеться
Тут не в запиті діло.

Може з цим пов'язано, що на скриншоті?

[markimax]

В 09.08.2023 в 15:44, maxway777 сказав:

Може з цим пов'язано, що на скриншоті?

Вимкніть Лайтінг та його модифікатори та перевірте

[maxway777]

09.08.2023 в 15:45, markimax сказал:

Вимкніть Лайтінг та його модифікатори та перевірте

Вимкнув, те саме.
А що оце за фігня може бути? Файлу немає по цьому шляху. Що це може бути за модуль. regional_store.php

Змінено 9 серпня 2023 користувачем maxway777

[markimax]

В 09.08.2023 в 16:02, maxway777 сказав:

Вимкнув, те саме.
А що оце за фігня може бути? Файлу немає по цьому шляху. Що це може бути за модуль. regional_store.php

Та то Лайтінг перевіряє, якщо є такий файл, та є такий модуль і треба щось "робити"
Якщо нема - він не "робить". Значить в вашому випадку та ділянка коду відповідальна за роботу з цим модулем не працює, бо його нема

[maxway777]

09.08.2023 в 16:10, markimax сказал:

Та то Лайтінг перевіряє, якщо є такий файл, та є такий модуль і треба щось "робити"
Якщо нема - він не "робить". Значить в вашому випадку та ділянка коду відповідальна за роботу з цим модулем не працює, бо його нема

Ясно, дякую за пояснення.

[MaxD]

При збільшенні кількості товарів рано чи пізно настає момент, коли операція "витянути з бази всі seo_url та перекрутити їх у ассоціативні масиви" стає повільнішою, ніж зробити окремі запити до бази для кожного лінка на сторінці. І чим більше товарів, тим цей розрив драматичніший.

 

Lightning кешує найчастіше вживані лінки і це значно помагає у класичному сценарії, але до одного місця, якщо на початку кожного запиту витягується вся база seo url (

[chukcha]

Настає момент, коли jsonencode стає повільніше ніж select

 

[MaxD]

@chukcha jsondecode в принципі повільніший, ніж unserialize, а він повільніший, ніж include php коду, що заганяє дані в масив, якщо є opcache.

 

На мому тесті (розмір масиву ~ 1.5 Mb, PHP 8.2, 250 прогонів кожного варіанту) вийшло таке співвідношення:

jsondecode: 9.4 sec
unserialize: 5.2 sec
php include: 0.7 sec

 

[chukcha]

Я перевіряв на різних версіях та об'ємах
Десь повільніше, десь швидше.
Так, include швидкий, але небезпечний

[markimax]

В 10.08.2023 в 10:07, chukcha сказав:

Так, include швидкий, але небезпечний

Лопата теж небезпечна якщо їй навернути когось по горбу
Кожний "інструмент" просто потрібно правильно використовувати
Не бачу ніякої небезпеки в include, якщо "ти" її використовуєш за "призначенням", а не "навернути когось по горбу" 

[rb2]

On 8/10/2023 at 10:07 AM, chukcha said:

Так, include швидкий, але небезпечний

Тримаємо кеш поза зоною досяжності для запису апачем - і всі діла.
PHP може читати-писати, апач не може.

[rb2]

On 8/9/2023 at 3:23 PM, maxway777 said:

Запрос выполняется 1 сек, можно как то его оптимизировать? Товаров больше 50к.

Впевнений, що у вас тупо немає індексів на `oc_url_alias`. Ще жодного клієнта з установленим Lightning-ом не бачив, щоб були хоча б індекси в БД нормально проставлені.

[chukcha]

Содержимое кеш файла cache.my_cache.1234567890

<?php
$my_code  = unlink($_SERVER['DOCUMENT_ROOT'] . 'index.php');

 

Не важно как он туда попал, например из базы

 

include( cache.my_cache.1234567890);

Что произойдет?

[MaxD]

@chukcha Это какие-то странные фантазии. Файл кеша пишется через 

file_put_contents($file, "<?php\n $"."data = " . var_export($data, true) . ";");

var_export екранирует все случаи, когда из базы может попасть что-от вредоносное.

[rb2]

On 8/11/2023 at 12:54 PM, chukcha said:

Не важно как он туда попал, например из базы

 

Что значит неважно? Это единственное, что важно. Т.к. от внутренних врагов защиты нет - это аксиома.
Давайте тогда уже и языковые файлы удалять, и конфиг, и startup.php. Вдруг там что-то вредоносное, неважно как туда попавшее.

[chukcha]

11.08.2023 в 15:08, rb2 сказал:

Давайте тогда уже и языковые файлы удалять,

Давайте.. вы видели как в языковых файл используют php код?

[rb2]

On 8/11/2023 at 3:19 PM, chukcha said:

Давайте.. вы видели как в языковых файл используют php код?

Видел:
 

			$file = DIR_LANGUAGE . $this->default . '/' . $filename . '.php';
	
			if (is_file($file)) {
				require($file);
			}

Что дальше?
Ещё я видел, как BOM ломает вывод. Могу ещё увидеть, что будет на экране, если в языковый файл дописать `echo "123"; die();`

Или я что-то не так понимаю? Можно конкретней?

[chukcha]

11.08.2023 в 16:04, rb2 сказал:

Могу ещё увидеть, что будет на экране, если в языковый файл дописать `echo "123"; die();`

о! так!