Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: zillow/zkafka
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v2.2.0
Choose a base ref
...
head repository: zillow/zkafka
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v2.2.1
Choose a head ref
  • 2 commits
  • 17 files changed
  • 2 contributors

Commits on Jun 30, 2025

  1. Rmpmp-995: Updated dependencies and improved error messages when miss… 

    …ing topic (#29)

Rmpmp-995: Updated dependencies and improved error messages when missing
topic name
---

The work loop uses a clientProvider.Reader() to get reader. The
`clientProvider` implementation of this method only returns an error
during misconfiguration. This is a permanent error and can't be
recovered. Work has been updated to mark Reader() error as permanent and
exit the work loop and return error.

Other changes:

1. recoverThenFail() were added in all zkafka tests. The reason being,
the use of containerized brokers during testing can lead to fragile test
behavior (seg faults have been observed from librdkafka). By recovering
in the tests, this can be better debugged since the entire test suite
doesn't fail.

---------

Co-authored-by: stewartboyd119 <stewartb@zillowgroup.com>
    @stewartboyd119 @stewartbzillow
    stewartboyd119 and stewartbzillow authored Jun 30, 2025
    Configuration menu
    Copy the full SHA
    ccc9efc View commit details
    Browse the repository at this point in the history

  2. Rmpm-995: fix release number and changelog (#30) 

    Rmpm-995: fix release number and changelog (updated deps based on github warnnings)

---------

Co-authored-by: stewartboyd119 <stewartb@zillowgroup.com>
    @stewartboyd119 @stewartbzillow
    stewartboyd119 and stewartbzillow authored Jun 30, 2025
    Configuration menu
    Copy the full SHA
    032980d View commit details
    Browse the repository at this point in the history
Loading