Далбеев Георгий / ИТМО DWS / Stage 1 #12
Conversation
![codeclimate[bot]](https://avatars.githubusercontent.com/in/9403?s=60&v=4){kind=small}
|
|
||
| Entry<MemorySegment> entry = new BaseEntry<>( | ||
| key, | ||
| MemorySegment.ofArray(request.getBody()) |
There was a problem hiding this comment.
Ключ провалидировали, но не хватает проверки тела на null. Иначе получается можно записать tombstone, тем самым удалив запись.
| return new Response(Response.BAD_REQUEST, Response.EMPTY); | ||
| } | ||
|
|
||
| Entry<MemorySegment> entry = dao.get(key); |
There was a problem hiding this comment.
Здесь и ниже в delete не обрабатываются возможные исключения DAO.
|
|
||
| public NewServer(ServiceConfig config) throws IOException { | ||
| super(configureServer(config)); | ||
| dao = new ReferenceDao(new Config(config.workingDir(), FLUSH_THRESHOLD)); |
There was a problem hiding this comment.
Лучше всего передавать ReferenceDao в конструктор NewServer и делать все это в NewService.
Так будет более гибко и полезно для юнит тестов, например.
|
|
||
| public class NewServer extends HttpServer { | ||
|
|
||
| private final ReferenceDao dao; |
There was a problem hiding this comment.
Почему конкретный класс, а не интерфейс?
| request = function() | ||
| local headers = {} | ||
| headers["Host"] = "localhost" | ||
| local id = math.random(1, 1000) |
There was a problem hiding this comment.
Очень мало ключей для нагрузочного тестирования. Можно ли доверять результатам?
There was a problem hiding this comment.
Да, я ошибся нужно было поставить выше чем rps. из-за этого скорее всего джит оптимизировал, и результаты не совсем корректные.
| ### Профилирование put | ||
| #### CPU | ||
| Библиотека one-nio затратила 20.81% на работу с запросами. | ||
| Операции записи ответа в сокет занимают всего 2.26%. |
There was a problem hiding this comment.
Почему 2.26, а не 14.03? А куда ушли остальные проценты?
| Операции записи ответа в сокет занимают всего 2.26%. | ||
| Операции сравнения MemorySegment занимают 11.31%. | ||
| #### Alloc | ||
| На созание MemorySegment затраивается 10.77%. |
There was a problem hiding this comment.
Во время какой операции происходят эти аллокации MemorySegment?
| Операции сравнения MemorySegment занимают 11.31%. | ||
| #### Alloc | ||
| На созание MemorySegment затраивается 10.77%. | ||
| Основне аллокации происходят на one-nio. |
There was a problem hiding this comment.
В процентном соотношении, на какие подоперации? парсинг заголовков, запросов и тп...
Хотелось бы увидеть более подробный аналитический отчет.
| На созание MemorySegment затраивается 10.77%. | ||
| Основне аллокации происходят на one-nio. | ||
|
|
||
| ### Профилирование get |
There was a problem hiding this comment.
при какой нагрузке - bad или normal, был снят профиль?
|
7/15 |
* stage1 * stage1 * stage1 * report stage1 * fix issues --------- Co-authored-by: georgiidalbeev <[email protected]> Co-authored-by: Roman Mushchinskii <[email protected]>
* stage1 * stage1 * stage1 * report stage1 * fix issues --------- Co-authored-by: georgiidalbeev <[email protected]> Co-authored-by: Roman Mushchinskii <[email protected]>
No description provided.