Skip to content

Merge Phase 1 follow-ups into master - #4

Open
coderconnoisseur wants to merge 11 commits into
masterfrom
sync-master-phase1
Open

coderconnoisseur wants to merge 11 commits into
masterfrom
sync-master-phase1

Conversation

@coderconnoisseur

Copy link
Copy Markdown
Owner

Brings the work from PR #2 (merged into `phase1-correctness` instead of `master`) into `master`:

  • env-var config for every service
  • `ride.unmatched` event from matching
  • assignment-service dedupe for `ride.matched`, `GET /rides/`, separate consumer group for `ride.unmatched`
  • epoch-stamped rewind-safe offset commits in `pkg/kafkaconsumer`
  • rider-simulator batched ticker

Head is `sync-master-phase1`: `origin/phase1-correctness` plus a merge of `origin/master`, so master's LICENSE (`29eb929`) and README line are kept. The merge had no conflicts.

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.

1 participant