Skip to content

Should stop offset backing store in Copycat Worker's stop method#232

Closed
Ishiihara wants to merge 1 commit into
apache:trunkfrom
confluentinc:bug-fix
Closed

Should stop offset backing store in Copycat Worker's stop method#232
Ishiihara wants to merge 1 commit into
apache:trunkfrom
confluentinc:bug-fix

Conversation

@Ishiihara

Copy link
Copy Markdown
Contributor

@ewencp @gwenshap This is a trivial bug fix

@Ishiihara Ishiihara changed the title Should stop offset backing store in Worker's stop method Should stop offset backing store in Copycat Worker's stop method Sep 23, 2015
@asfbot

asfbot commented Sep 23, 2015

Copy link
Copy Markdown

kafka-trunk-git-pr #492 FAILURE
Looks like there's a problem with this pull request

@ewencp

ewencp commented Sep 23, 2015

Copy link
Copy Markdown
Contributor

FYI, this fix also appears in #202 with updated tests (WorkerTest was missing mocks/expectations on OffsetBackingStore).

@Ishiihara Ishiihara closed this Sep 23, 2015
@ewencp
ewencp deleted the bug-fix branch December 27, 2016 17:21
jsancio pushed a commit to jsancio/kafka that referenced this pull request Aug 6, 2019
efeg pushed a commit to efeg/kafka that referenced this pull request Jan 29, 2020
wyuka pushed a commit to wyuka/kafka that referenced this pull request Jan 21, 2022
…thAuthenticationFailure (apache#232)

Co-authored-by: Ke Hu <kehu@kehu-mn2.linkedin.biz>
davide-armand pushed a commit to aiven/kafka that referenced this pull request Dec 1, 2025
Using fixed port (7800) with the 10 port range by default causes clashes
on concurrent tests.
The configuration passed is the default one but with bind_port=0 to use
any ephemeral port.
jeqo added a commit to aiven/kafka that referenced this pull request Jan 16, 2026
Using fixed port (7800) with the 10 port range by default causes clashes
on concurrent tests.
The configuration passed is the default one but with bind_port=0 to use
any ephemeral port.
fvaleri pushed a commit to fvaleri/kafka that referenced this pull request Jul 21, 2026
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