Skip to content

Comments

feat: Allow indirect replication#32233

Merged
patriknw merged 1 commit intomainfrom
wip-res-indirect-patriknw
Nov 23, 2023
Merged

feat: Allow indirect replication#32233
patriknw merged 1 commit intomainfrom
wip-res-indirect-patriknw

Conversation

@patriknw
Copy link
Contributor

  • it was only using events from the origin replica
  • this allows indirect replication, e.g. A writes e1, replicates to B, and C receives e1 from B
  • will be needed for some edge topologies
  • also remove the check that replica of received event is included in allReplicas
  • there will be a corresponding change in Projection gRPC

See akka/akka-projection#1069

* it was only using events from the origin replica
* this allows indirect replication, e.g. A writes e1, replicates to B,
  and C receives e1 from B
* will be needed for some edge topologies
* also remove the check that replica of received event is included in
  allReplicas
* there will be a corresponding change in Projection gRPC
@patriknw patriknw force-pushed the wip-res-indirect-patriknw branch from 5b0a49b to 216da06 Compare November 14, 2023 15:09
Copy link
Member

@pvlugter pvlugter left a comment

Choose a reason for hiding this comment

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

LGTM

@patriknw patriknw merged commit 3dbcf1c into main Nov 23, 2023
@patriknw patriknw deleted the wip-res-indirect-patriknw branch November 23, 2023 09:44
@patriknw patriknw added this to the 2.9.1 milestone Nov 23, 2023
He-Pin pushed a commit to He-Pin/akka that referenced this pull request Jan 7, 2024
* it was only using events from the origin replica
* this allows indirect replication, e.g. A writes e1, replicates to B,
  and C receives e1 from B
* will be needed for some edge topologies
* also remove the check that replica of received event is included in
  allReplicas
* there will be a corresponding change in Projection gRPC
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.

3 participants