Skip to content
This repository was archived by the owner on May 12, 2026. It is now read-only.

Небольшая оптимизация запросов#224

Open
Aligator77 wants to merge 2 commits into
segmentstream:masterfrom
Aligator77:master
Open

Небольшая оптимизация запросов#224
Aligator77 wants to merge 2 commits into
segmentstream:masterfrom
Aligator77:master

Conversation

@Aligator77

Copy link
Copy Markdown

убрал блок then, который не нужен

Убрал then, затем, что он только тормозит процесс отправки данных(в ответ мы все равно ничего не получим)
@KirillTemnov

Copy link
Copy Markdown
Contributor

Привет, ты удалил код который логически связан с закомментированным блоком https://github.com/segmentstream/digital-data-manager/pull/224/files#diff-12a81aae7db95a321da68f9d7f8d4c51R373

Не могу принять PR

@Aligator77

Aligator77 commented Nov 7, 2019

Copy link
Copy Markdown
Author

Привет, ты удалил код который логически связан с закомментированным блоком https://github.com/segmentstream/digital-data-manager/pull/224/files#diff-12a81aae7db95a321da68f9d7f8d4c51R373

Не могу принять PR

Так если ваш сервис не возвращает никакого ответа, то удаленный блок совсем не имеет смысла, а лишь задерживает загрузку страницы.
Для примера, с сайта отправляется 5 запросов, 3 из них скачивают контент по ~200ms, в итоге страница рендерится на 0,5 сек медленнее, чем могла бы.
ссылка на скриншот времени скачивания контента - https://yadi.sk/i/-rHrdlIKvTfqLw

@KirillTemnov

Copy link
Copy Markdown
Contributor

Решение: закомментировать этот код вместо удаления.

@KirillTemnov

Copy link
Copy Markdown
Contributor

Если речь идёт про метрики, то лучше делать сравнение "до" и "после" на https://www.webpagetest.org/ или аналогичных инструментах.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants