Для сеттера типа:

public void setEntityName(String entityName) {
  this.entity = loadFromDBByName(entityName);
  this.entityName = entityName;
}

На код ревью было сделано замечание, что сеттер слишком сложный и надо переименовать метод в что-то типа loadEntityByName. На мой же взгляд это нормально, что сеттер что-то делает помимо чисто установки значения. Для того и задумывалось использование оберток надо членами класса, чтобы выполнять в них необходимую дополнительную работу. Или я что-то пропустил изучая ООП?