имхо, чем меньше данных содержит ивент - тем лучше.
но, например, ивент об изменение сущности вполне логично может содержать старые и новые значения.
Я видел одну системку, там все ивенты были типа
AbstractEntityEvent - сожержит все поля сущности
EntityCreatedEvent extends AbstractEntityEvent
EntityUpdatedEvent extends AbstractEntityEvent - содержит кроме текущего состояния еще и все предыдущее
EntityDeletedEvent extends AbstractEntityEvent
в итоге хендлерам не нужно было вообще никуда ходить за данными самой ентити, все было тут, но сильно трафик рос в системе =) большие ивенты и их было много.
хз, мне не показалось это очень удобным, по правде говоря