Кислов Данил ИТМО ФИТиП HW1 #3
Conversation
![codeclimate[bot]](https://avatars.githubusercontent.com/in/9403?s=60&v=4){kind=small}
src/main/java/ru/vk/itmo/test/kislovdanil/service/DatabaseServiceFactory.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/vk/itmo/test/kislovdanil/dao/iterators/MergeIterator.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/vk/itmo/test/kislovdanil/service/DatabaseHttpServer.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/vk/itmo/test/kislovdanil/service/DatabaseHttpServer.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/vk/itmo/test/kislovdanil/dao/PersistentDao.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/vk/itmo/test/kislovdanil/service/DatabaseHttpServer.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/vk/itmo/test/kislovdanil/service/DatabaseServiceFactory.java
Show resolved
Hide resolved
| @Override | ||
| public CompletableFuture<Void> stop() throws IOException { | ||
| dao.close(); | ||
| httpServer.stop(); |
There was a problem hiding this comment.
Что будет, если после закрытия dao придут новые запросы и попытаются обратиться к dao? Возможно, мы не хотим думать про такие случаи и хотим от них обезопаситься
There was a problem hiding this comment.
И это нетривиальный вопрос, в прошлом семестре мы жили с предположением, что после закрытия методы dao никто не вызывает. Зависит от реализации dao, а реализация со временем может меняться. Так что, видимо, UB
src/main/java/ru/vk/itmo/test/kislovdanil/service/DatabaseHttpServer.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/vk/itmo/test/kislovdanil/service/DatabaseHttpServer.java
Show resolved
Hide resolved
src/main/java/ru/vk/itmo/test/kislovdanil/service/DatabaseHttpServer.java
Show resolved
Hide resolved
src/main/java/ru/vk/itmo/test/kislovdanil/report/stage1/get11000full
Outdated
Show resolved
Hide resolved
src/main/java/ru/vk/itmo/test/kislovdanil/report/stage1/report.md
Outdated
Show resolved
Hide resolved
| # Отчёт о нагрузочном тестировании | ||
| ## Этап 1 | ||
|
|
||
| Опытным путём установлена *точка разладки* ~5.000 RPS. |
There was a problem hiding this comment.
Это точка разладки конкретно get запросов или для put тоже? Учитывая специфику lsm, put запросы должны держать больший rps.
|
|
||
| #### Флеймграфы для PUT запросов | ||
| ##### CPU | ||
|  |
There was a problem hiding this comment.
Вижу в профиле столбец справа "run", есть идеи что это за процесс и откуда он?
|  | ||
|
|
||
| ##### Allocations | ||
|  |
There was a problem hiding this comment.
Довольно много аллокаций происходят вне SelectorThread. Есть идеи, откуда они?
|
|
||
| /* Binary search in summary and index files. Returns index of least record greater than key or equal. | ||
| Returns -1 if no such key */ | ||
| long findByKey(MemorySegment key) { |
There was a problem hiding this comment.
В этом методе происходит ~70-80% аллокаций вашего кода. Как можно уменьшить их число?
И было бы очень круто применить эту оптимизацию, чтобы при выполнении следующих alloc профиль был почище.
| } | ||
|
|
||
| @Path(ENTITY_ACCESS_URL) | ||
| public Response handleEntityRequest(Request request, |
There was a problem hiding this comment.
Еще, как говорил Вадим на паре, стоит добавить обработку непредвиденных исключений на случай серверных ошибок
| } | ||
|
|
||
| @Override | ||
| public CompletableFuture<Void> stop() throws IOException { |
There was a problem hiding this comment.
Думаю start, stop методы стоит сделать synchoronized, например, если два потока зайдут в stop и dao.close(), они могут дважды закрыть арену, что уже эксепшен.
* Stage 1 * linter fixes * linter fixes * Правки в отчёте после ревью --------- Co-authored-by: kislovda <kislovda@mos-team.ru> Co-authored-by: Alexey Shik <58121508+AlexeyShik@users.noreply.github.com>
* Stage 1 * linter fixes * linter fixes * Правки в отчёте после ревью --------- Co-authored-by: kislovda <kislovda@mos-team.ru> Co-authored-by: Alexey Shik <58121508+AlexeyShik@users.noreply.github.com>
Отчет лежит в src/main/java/ru/vk/itmo/test/kislovdanil/report/stage1/report.md