Stage 1: Emelyanov Vitaliy, SPbSTU #15
Conversation
![codeclimate[bot]](https://avatars.githubusercontent.com/in/9403?s=60&v=4){kind=small}
There was a problem hiding this comment.
Есть один серьезный момент - это отсутствие обработки исключений
Также отсутствие вывода wrk не позволяет проверить полученные выводы и корректность нагрузки. Убедительная просьба в следующих этапах это учесть, чтобы, как минимум, не терять баллы за этап )
10 баллов
| public static final byte[] EMPTY_BODY = new byte[0]; | ||
| private final ReferenceDao dao; | ||
|
|
||
| public DhtServer(ServiceConfig config) throws IOException { |
There was a problem hiding this comment.
Думаю, не очень хорошо, что конфиг сервиса просочился на уровень сервера. Было бы лучше, если бы сервис разобрал данный конфиг, на его основе сформировал бы конфиги зависимостей и инстанцировал их с этими конфигами
| import static one.nio.http.Request.METHOD_PUT; | ||
|
|
||
| public class DhtServer extends HttpServer { | ||
| public static final byte[] EMPTY_BODY = new byte[0]; |
|
|
||
| public DhtServer(ServiceConfig config) throws IOException { | ||
| super(createConfig(config)); | ||
| dao = new ReferenceDao(new Config(config.workingDir(), 1 << 24)); |
| } | ||
|
|
||
| @RequestMethod(METHOD_GET) | ||
| @Path("/v0/entity") |
There was a problem hiding this comment.
Используется в нескольких местах, можно было бы в константу выделить
| if (isKeyIncorrect(id)) { | ||
| return new Response(Response.BAD_REQUEST, EMPTY_BODY); | ||
| } | ||
| Entry<MemorySegment> entry = dao.get(keyFor(id)); |
There was a problem hiding this comment.
Для всех методов отсутствует обработка исключений от DAO
Что, кстати, будет при выбросе исключения?) Автоматом 500 ответ или, может, отсутствие ответа?)
|
|
||
| public class DhtService implements Service { | ||
| private final ServiceConfig serviceConfig; | ||
| private DhtServer server; |
There was a problem hiding this comment.
Вероятно, будет достаточно HttpServer/Server
| # Отчет по модулю 1 | ||
|
|
||
| ## Наполнение | ||
| Для наполнения был использован следующий скрипт на lua |
There was a problem hiding this comment.
В отчёте необходимо приводить вывод wrk, чтобы ревьюер мог проверить полученные выводы
|
|
||
| [Аллокации на чтение](alloc_read.html) | ||
|
|
||
| Все аллокации выполняются исключительно для возврата/получения значения |
There was a problem hiding this comment.
Еще же видно, что часть сэмплов (не такая маленькая) - это аллокации при парсинге запроса силами one-nio
В принципе, возможности на них повлиять особой нет. Но можно было упомянуть, что они есть
|
Oops
|
@GenryEden хорошо бы отвечать на комментарии |
|
я прошу прощения, проглазел. в беседе в вк обсуждали, что это флакало у кого-то другого, кого замержили в мейн, сейчас не вопсроизводится |
No description provided.