Open
Conversation
spajic
approved these changes
Sep 3, 2020
Collaborator
spajic
left a comment
There was a problem hiding this comment.
Круто, спасибо за интересные кейсы!
Вы конечно вихрем пронеслись через этот курс 🌪️
Надеюсь, что было полезно!
| ### Проект #1 | ||
|
|
||
| В этом задании вам нужно написать `case-study` о том как вы применили знания, полученные на курсе, к своим проектам. | ||
| В этот case study хотел бы включить кейсы из разных проектов, в которых приходилось принимать участие. Один из таких проектов интересен тем, что большАя часть проекта просто [зашифрована](https://www.rubyencoder.com/). Это было сделано предыдущими разработчиками для защиты интеллектуальной собственности (вместо NDA). Приходилось смотреть в логи и по ним понимать, что примерно происходит внутри. |
| ## Как сдать задание | ||
|
|
||
| Сделайте `PR` в этот репозиторий с вашим `case-study`. | ||
| 4) Прогнал тесты вместе со встроенным профилировщиком `rspec spec --profile`. Он показал несколько самых медленных тестов, в которых была одна общая проблема: создавался объект вне всех `context`, а использовался лишь в одном или нескольких. После переноса `let` в нужный `context` прирост скорости был очень значительный (в некоторых случаях в два раза с 50 до 23 сек). No newline at end of file |
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.
Проект 1
В этот case study хотел бы включить кейсы из разных проектов, в которых приходилось принимать участие. Один из таких проектов интересен тем, что большАя часть проекта просто зашифрована. Это было сделано предыдущими разработчиками для защиты интеллектуальной собственности (вместо NDA). Приходилось смотреть в логи и по ним понимать, что примерно происходит внутри.
На этом проекте не было мониторинга и при этом были большие проблемы с оптимизацией.
Самой большой точкой роста был N+1 запрос для построения меню. Меню рисовалось на всех страницах приложения, так что это была самая жирная точка роста и от нее надо было избавляться в первую очередь. До кода я добраться не мог. Пришлось закэшировать участок кода во вьюхе.
Меню меняется редко, поэтому закешировать его - не проблема.
Был момент, когда пришлось захардкодить это меню и вставить чистый html. Это было приемлимо, но решение продержалось недолго. Но как вариант можно использовать.
На этом же проекте был автокомплит (в курсе тоже этот кейс упоминался, но я реализовал ограничение от трех букв на фронте, так что запросы на бэк не шли вообще, пока пользователь не ввел три буквы).
Еще, что заметно прибавило скорости, так это выставление index'ов на все внешние ключи. Искал отсутствующие индексы с помощью гема (не помню название).
На проекте я работал один и честно говоря не было времени всерьез заниматься профилированием.
Проект 2
Проект достаточно давно разрабатывается. С перфомансом на проекте все хорошо. На этом проекте есть мониторинги: Grafana для железа, PgHero для SQL. На вскидку предполагаю, что на проекте есть много N+1, которые мне еще предстоит оптимизировать. В тестах - каскады в фабриках.
Это было до 6 рельсов, сейчас переходим на изкоробочное решение
upsert_all/insert_all.#upsert_allВ качестве профилировщика использовался
memory_profiler. Объемы потребляемой памяти в сравнении с кастомным классом сократились в 4x. Эта оптимизация еще не доведена до прода.Первым делом обратился к PgHero, он не показал что запрос является точкой роста. Но в логах я заметил лишние запросы типа
Exist?(запрос в логах был не во всех фильтрах). Я начал искать, где в запросе есть проверки на существование записей и нашел#any?. Добавил#loadи проблема исчезла. Понимаю что это скорее костыль и проблему надо решать по другому (скоре переписать кейс на query-object стиль и оставить что-то одно).Это небольшая точка роста на любом проекте, это просто хорошая практика :)
Перечитывая доки, наткнулся на такой кейс. В итоге написал кастомный коп для этого.
Прогнал тесты вместе со встроенным профилировщиком
rspec spec --profile. Он показал несколько самых медленных тестов, в которых была одна общая проблема: создавался объект вне всехcontext, а использовался лишь в одном или нескольких. После переносаletв нужныйcontextприрост скорости был очень значительный (в некоторых случаях в два раза с 50 до 23 сек).