Skip to content
This repository has been archived by the owner on Jan 13, 2023. It is now read-only.

Вопрос по реализации addRoleToAllUsers в задании user-service #9

Open
yegorvin opened this issue Jan 9, 2023 · 0 comments

Comments

@yegorvin
Copy link

yegorvin commented Jan 9, 2023

Здравствуйте.

В вашем готовом примере addRoleToAllUsers реализован так:

@Override public void addRoleToAllUsers(RoleType roleType) { List<User> users = entityManager.createQuery("select u from User u left join fetch u.roles", User.class) .getResultList(); users.stream() .filter(user -> hasNoRole(user, roleType)) .forEach(user -> user.addRole(Role.valueOf(roleType))); }

У меня возникло 2 вопроса:

  1. зачем в запросе "... fetch u.roles", spring и так их вытянет или это оптимизация чтобы уменьшить лишний select из базы?

  2. users.stream().filter(user -> hasNoRole(user, roleType)) .. Role на момент добавления в HashSet имеет null id и будет дубликат без этой проверки. В spring для такого случая нет готового решения, чтобы избежать дубликатов?

Спасибо.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant