Conversation
EugeneLenkevich
left a comment
EugeneLenkevich
left a comment
There was a problem hiding this comment.
Привет, Вячеслав! Спасибо за PR.
Интересная, конечно, работа, в целом все ок, предложу попробовать типизировать результат запроса (сложить в dto)
List<Object[]> averageResult
но не настаиваю, если времени нет то шли обратно как есть :)
| FROM max_weights m | ||
| GROUP BY m.event_id | ||
| """) | ||
| List<Object[]> findWeightSumListByEventIdList( |
There was a problem hiding this comment.
Object[]
А зачем кстати массив, не получиться ли в DTO сложить результат?
There was a problem hiding this comment.
Я посмотрел, и, конечно, лучше использовать DTO, но мне, если честно, уже лень. Я сейчас не на основном компьютере и тут докер не очень корректно работает. Сложно тестировать
There was a problem hiding this comment.
Вообще, ты крутой, хорошие работы, интересно проверять
There was a problem hiding this comment.
Спасибо за ревью и хорошие советы. Было приятно с вами взаимодействовать
EugeneLenkevich
left a comment
EugeneLenkevich
left a comment
There was a problem hiding this comment.
Привет, ок, пусть будет так :)
3 participants
No description provided.