Андрей Смирнов ИТМО КТ stage6 #219
Conversation
![codeclimate[bot]](https://avatars.githubusercontent.com/in/9403?s=60&v=4){kind=small}
src/main/java/ru/vk/itmo/test/smirnovandrew/ChunkTransformUtility.java
Outdated
Show resolved
Hide resolved
AlexeyShik
requested review from
daniil-ushkov
and removed request for
AlexeyShik
| try { | ||
| session.write(MyServerUtil.CHUNKED_HEADERS, 0, MyServerUtil.CHUNKED_HEADERS.length); | ||
| for (var it = dao.getEntriesFromDao(start, end); it.hasNext(); ) { | ||
| var sessionBody = MyServerUtil.getSessionBody(it.next()); |
There was a problem hiding this comment.
Правда ли, что если в записи лежит очень большое value, то ты будешь писать его в сокет целиком?
There was a problem hiding this comment.
Да, правда, а надо поправить это?
There was a problem hiding this comment.
Уточнил у коллег, вам это обрабатывать не нужно, так что не буду за это снижать
| @@ -0,0 +1,34 @@ | |||
| # Отчет о тестировании | |||
There was a problem hiding this comment.
Здесь было бы неплохо проверить, что происходит при выгрузке всех ключей из базы одним range запросом (кажется это было в задании).
|
|
||
| Анализ ALLOC: | ||
|  | ||
| Большее количество аллокаций приходится на конвертацию из String в |
There was a problem hiding this comment.
Звучит как то, что можно оптимизировать! Это просто замечание, на баллы не повлияет.
|
|
||
| Анализ CPU: | ||
|  | ||
| Наибольшая нагрузка как и в предыдущих частях приходится на ожидание |
There was a problem hiding this comment.
Вообще говоря это нехорошо. Sleep системных потоков нужно сводить к нулю, так как это бесполезная трата времени. Для этого и существуют future, корутины, асинхронность и т.п.
Это также просто замечание, не влияющее на баллы.
|
Готов поставить 14 баллов. |
|
Просьба порезолвить конфликты. |
No description provided.