Task - 3. #85
Open
Task - 3. #85
Conversation
This file contains 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
pochkuntaras
changed the title
spajic
approved these changes
Jun 18, 2022
| <%#= render trip.bus.services %> | ||
| </ul> | ||
| <% end %> | ||
| <%= '=' * 52 %> |
There was a problem hiding this comment.
хех, первый раз такое представление делимитера в этом ДЗ, лайк
There was a problem hiding this comment.
Если удалить паршлы и заинлайнить их, то рендерится быстрее конечно
Но есть возможность использовать паршлы и не убивать перформанс: рендеринг коллекций
Там можно даже шаблон делимитера задать аргументом: https://guides.rubyonrails.org/layouts_and_rendering.html#spacer-templates
| nil | ||
| end | ||
|
|
||
| def add_item(collection, item_name) |
| s = Service.find_or_create_by(name: service) | ||
| services << s | ||
| Benchmark.bm do |x| | ||
| x.report('Clearing database') do |
There was a problem hiding this comment.
Очистку данных и загрузку лучше бы ещё в транзакцию завернуть
| @@ -0,0 +1,176 @@ | |||
| #№ Оптимизация импорта данных | |||
| Установил `gem 'annotate'` для того, что бы увидеть зависимости, поля и индексы таблиц. | |||
| Creating data 35.507971 0.369157 35.877128 ( 44.157531) | ||
| ``` | ||
|
|
||
| Из результатов видно, что мы достигли желаемого результата в бюджет в одну минуту. |
|
|
||
| - Из анализа запросов `rack-mini-profiler` видно, что есть проблемы с индексами таблиц. | ||
| Для анализа проблем с индексами установим `gem 'pghero'` и `gem 'pg_query'`. | ||
| Обозримых проблем инструмент не дал. |
There was a problem hiding this comment.
Обычно в этом ДЗ pghero прям показывает все индексы, которые стоит создать
Возможно не было достаточно данных у него для анализа / или что-то не заработало
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.
No description provided.