Make KafkaJournalWriter support a list as 'brokers'.
ClosedPublic
Actions

Authored by vlorentz on Sep 18 2019, 5:15 PM.

Details

Reviewers

olasd

Group Reviewers

Reviewers

Diff Detail

Repository

rDJNL Journal infrastructure

Branch

kafka-servers-list

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 7842
Build 11281: tox-on-jenkins	Jenkins
Build 11280: arc lint + arc unit

Event Timeline

vlorentz created this revision.Sep 18 2019, 5:15 PM

vlorentz added a child revision: D2002: Poll after each call to produce(), as recommended by the docs..

Build is green
See https://jenkins.softwareheritage.org/job/DJNL/job/tox/255/ for more details.

Harbormaster completed remote builds in B7838: Diff 6730.Sep 18 2019, 5:18 PM

olasd accepted this revision.Sep 18 2019, 5:23 PM

This revision is now accepted and ready to land.Sep 18 2019, 5:23 PM

rebase

Harbormaster failed remote builds in B7842: Diff 6734!Sep 18 2019, 5:29 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DJNL/job/tox/259/
See console output for more information: https://jenkins.softwareheritage.org/job/DJNL/job/tox/259/console

Landed as ebe0a5cb8a9a.

Revision Contents
Changeset List

Path

Size

swh/

journal/

tests/

test_kafka_writer.py

4 lines

writer/

kafka.py

5 lines

Diff 6734

View Options

swh/journal/tests/test_kafka_writer.py