Skip to content

feat: add ability to reproject broken entities - #37

Open
felixgehrmann wants to merge 1 commit into
masterfrom
feat/projector_reproject
Open

feat: add ability to reproject broken entities#37
felixgehrmann wants to merge 1 commit into
masterfrom
feat/projector_reproject

Conversation

@felixgehrmann

Copy link
Copy Markdown

Entitites with incorrect version or that are downright missing can complety block eventhandling.
Developers have to fix this manually until now.

@felixgehrmann
felixgehrmann requested a review from MaxBreida July 23, 2026 10:59

// reprojectBrokenEntity projects an entity from zero to current event version minus 1.
// Useful for when projections are broken for whatever reason, but not very performant.
func (h *EventHandler) reprojectBrokenEntity(ctx context.Context, id uuid.UUID, event eh.Event, causingErr error) (eh.Entity, error) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

haben wir denn hier irgendwann auch mal ne ausstiegsklausel, sodass wir irgendwann mal aufhören mit dem Versuch hiervon?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Er probiert es ja nur einmal pro Run des EventHandlers? Wir haben glaube ich keinen allgemeinen Marker hier verfügbar, der sagt, dass es komplett im Arsch ist.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Joa aber im Zweifel halt für immer 😄

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aber meine steile These ist halt, dass nen Retry auch nix ändern wird, wenn das Reprojecten auch failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants