Homework1. Тяпуев Дмитрий. Магистратура. Политех #21
Conversation
![codeclimate[bot]](https://avatars.githubusercontent.com/in/9403?s=60&v=4){kind=small}
src/main/java/ru/vk/itmo/test/tyapuevdmitrij/ServerImplementation.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/vk/itmo/test/tyapuevdmitrij/ServerStarter.java
Outdated
Show resolved
Hide resolved
| return new Response(Response.ACCEPTED, Response.EMPTY); | ||
| } | ||
|
|
||
| public void closeDAO() { |
There was a problem hiding this comment.
странно, что dao открывает один класс, а закрывает другой. При этом сначала вызывается метод close у server'a, а затем сразу же вызывается метод close у dao.
| if (id == null || id.isEmpty()) { | ||
| return new Response(Response.BAD_REQUEST, Response.EMPTY); | ||
| } | ||
| MemorySegment key = MemorySegment.ofArray(id.getBytes(StandardCharsets.UTF_8)); |
There was a problem hiding this comment.
здесь и ниже по коду никак не обрабатываются исключения dao
|
|
||
| public class ServiceImplementation implements ru.vk.itmo.Service { | ||
|
|
||
| private ServerImplementation server; |
There was a problem hiding this comment.
сначала поля с final, затем уже без final.
| return new Response(Response.BAD_REQUEST, Response.EMPTY); | ||
| } | ||
| Entry<MemorySegment> entry = new BaseEntry<>(MemorySegment.ofArray(id.getBytes(StandardCharsets.UTF_8)), | ||
| MemorySegment.ofArray(request.getBody())); |
There was a problem hiding this comment.
request.getBody() никак не проверяется. Оно может быть null, что, кажется, не совсем ожидается от метода PUT
Такое поведение - это уже DELETE
| (тут все занимает MemorySegment.asSlice). | ||
|
|
||
| ## Вывод | ||
| По результатам экспериментов выяснилось, что база может гораздо лучше принимать данные, чем их искать. Необходима |
There was a problem hiding this comment.
"база может гораздо лучше принимать данные, чем их искать"
- Непонятно, что такое "лучше". Вероятно, на чтение база держит меньше rps, чем максимальное кол-во rps при записи
- Нельзя было такой вывод сделать без нагрузочного тестирования? Модель LSM заточена на то, чтобы принимать большое кол-во записей.
| 2. 3000 запросов в секунду на протяжении 1 минуты: У сервера появились проблемы. На последних 10 персентиль | ||
| резко возрастает время. Однако все время пока остается в пределах 100мс. | ||
|  | ||
| 3. 10000 запросов в секунду на протяжении 1 минуты: Сервер перестал справляться |
There was a problem hiding this comment.
А точку разладки не нашли? Из вашего анализа непонятно, какой максимальный rps на чтение способна держать ваша реализация.
| 3. 10000 запросов в секунду на протяжении 1 минуты: У сервера появились проблемы. На последнем персентиль | ||
| резко возрастает время. Однако все время пока остается в пределах 100мс. | ||
|  | ||
| 4. 30000 запросов в секунду на протяжении 1 минуты: Сервер перестал справляться. |
There was a problem hiding this comment.
А точку разладки не нашли? Из вашего анализа непонятно, какой максимальный rps на запись способна держать ваша реализация.
* added my DAO and http rest api * some fixes for codeclimate * all done --------- Co-authored-by: Vladislav Zybkin <[email protected]>
No description provided.