Нет, у вас не будет бесконечного цикла.
Я не думаю, что на этом уровне есть хорошие или плохие практики.Иногда лучше использовать LAZY, потому что это мешает вам загружать ненужные вам объекты.Иногда лучше использовать EAGER, так как он будет препятствовать запуску нескольких запросов Hibernate, если вам нужны объекты.
При программировании микросервисов я всегда использую EAGER (и т. Е. Ограниченный контекстный агрегат).Для монолита, может быть, ленивый мудрее.