Open
Conversation
added 2 commits
February 24, 2025 11:34
spajic
approved these changes
Feb 28, 2025
| @@ -1 +1 @@ | |||
| 3.3.1 | |||
| 3.4.1 | |||
| variable :ALGOLIASEARCH_SEARCH_ONLY_KEY, :String, default: only_in_test | ||
|
|
||
| # NewRelic | ||
| variable :NEW_RELIC_LICENSE_KEY, :String, default: "Optional" |
| end | ||
|
|
||
| gem 'newrelic_rpm' | ||
| gem 'newrelic-infinite_tracing' |
| # Доп подкючение local_production | ||
|
|
||
| При создании local_production окружении и использовании его, видны следующие улучшения максимально приближённые к продовому окруэениею. | ||
| - время загрузки посторной страницы занимает теперь не 588.4 ms, а 129 ms |
|
|
||
| При создании local_production окружении и использовании его, видны следующие улучшения максимально приближённые к продовому окруэениею. | ||
| - время загрузки посторной страницы занимает теперь не 588.4 ms, а 129 ms | ||
| - при нагрузки через `ab -n 100 -c 5 127.0.0.1:3000/`, уменьшилось в 10 раз среднее время по всем показателям |
|
|
||
| services: | ||
| agent: | ||
| container_name: newrelic-infra |
Collaborator
There was a problem hiding this comment.
по идее это не особо обязательно; достаточно поставить гем, указать название приложения, ключ и всё
это уже для мониторинга самого компьютера/докера кмк, не уверено что это в данном случае нужно
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Case Study
Подготовка
Оптимизация
Сразу для того чтобы сравнить решил сравнивать вывод в NewRelic и RMP(по подробности данным всё же RMP более удобный, меньше повторяющейся информации и больше подробностей).
Получил что при первой загруку страница запускалась где-то 3252.06 ms, при повторной ускорилось до 1300 - 1600.1 ms, из-за разных кэшей браузера и уже настроенных кэшей приложения.
через RMP видно что проблема рендерингом main_stories_feed, а точнее _single_story
Details
при нагрузке через ab, сразу отваливается по таймауту, результаты получит не получилось.
Details
Посмотревчто можно сделать, и учитывая совет из задачи мы понимаем что нам позволительно закэшировать
<%= render "articles/single_story", story: story %>.Проверим эту гипотезу, добавляем кэширование
Как результат первая загрузка не поменялас в среднем как и было 3252.06 ms, но при этом любая повторная загрузка уже выполняется в среднем за 588 ms. Что довольно силь что в 2.2 быстрее чем без использования кэшей. Так же успешно удалось назузить через
ab -n 100 -c 5 127.0.0.1:3000/Details
Заключение
Как вердикт проверка теории о использовании кэшей для
articles/single_storyпринесла свои результаты и ускорила повторные загрузки страницы в 2 раза.Доп подкючение local_production
При создании local_production окружении и использовании его, видны следующие улучшения максимально приближённые к продовому окруэениею.
ab -n 100 -c 5 127.0.0.1:3000/, уменьшилось в 10 раз среднее время по всем показателямбыло
стало
Details