Conversation
EugeneLenkevich
left a comment
EugeneLenkevich
left a comment
There was a problem hiding this comment.
Привет, Вячеслав! Спасибо за PR.
Отличная работа! :)
Сервисы в эврике регистрируются, супер, все вроде на месте, отметил
пару небольших моментов, посмотри комментарии, пожалуйста :)
В первую очередь логирование бы поправить, хотелось бы какие то идентификаторы в логах увидеть чтобы запросы идентифицировать. Иначе потом не разобраться с тем что произошло на проде в случае какого либо инцидента
| private final CategoryAdminService categoryAdminService; | ||
|
|
||
| @PostMapping | ||
| public ResponseEntity<CategoryDto> addCategory( |
There was a problem hiding this comment.
А во эти обертки, ResponseEntity, нужны ли, может лучше аннотациями задать нужные коды, нужен ли нам этот лишний код :)
| @Service | ||
| @RequiredArgsConstructor | ||
| @Slf4j | ||
| @Transactional |
There was a problem hiding this comment.
Давай все таки транзакции повесим на тех методах, которые меняют данные, а томы селекты гоняем в транзакциях только ресурсы лишние расходуем. То есть для операций чтения в общем случае транзакции не нужны :)
|
|
||
| @Override | ||
| public CommentDto approveComment(Long comId) { | ||
| log.info("approveComment - invoked"); |
There was a problem hiding this comment.
Добавить бы в логи идентификаторы, comId в данном случае. Потом же в логе не будет понятно какая строка к какому запросу или объекту относится и не будет возможности собрать всю цепочку вызовов по конкретному объекту. В реальном приложении у нас будет множество запросов одновременно :)
EugeneLenkevich
left a comment
EugeneLenkevich
left a comment
There was a problem hiding this comment.
Теперь все отлично, спасибо :)
3 participants
No description provided.