Skip to content

chore: use :broadway v1.3 over forked copy#3680

Open
amokan wants to merge 1 commit into
mainfrom
chore/update-broadway-lib
Open

chore: use :broadway v1.3 over forked copy#3680
amokan wants to merge 1 commit into
mainfrom
chore/update-broadway-lib

Conversation

@amokan

@amokan amokan commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

This is one we'll want to monitor on staging/canary, but this seems like a solid and low-risk move.

We've been using our own fork of Broadway for some time, but I noticed that the pinned branch we were on is actually in the canonical Broadway repo and is included in the latest v1.3 release.

Delta between our forked/pinned branch and Broadway v1.3 can be found here - Logflare/broadway@fix/batcher-fullsweep-after...elixir-broadway:broadway:v1.3.0

Here is Ziinc's merged PR into Broadway from the pinned version we were on.

Why Switch To v1.3?

A couple standout items that could be beneficial to us in one way or another are:

@djwhitt djwhitt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed the changelog. Looks low risk, and I really want that process labeling capability.

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