Skip to content

debug: add join/rejoin diagnostic logging#2855

Merged
dpkp merged 1 commit intomasterfrom
claude/flakey-consumer-tests
Apr 11, 2026
Merged

debug: add join/rejoin diagnostic logging#2855
dpkp merged 1 commit intomasterfrom
claude/flakey-consumer-tests

Conversation

@dpkp
Copy link
Copy Markdown
Owner

@dpkp dpkp commented Apr 11, 2026

Add temporary debug logging to investigate consumer rejoin flakiness
where _handle_join_success runs (logging "Enabling heartbeat thread")
but _on_join_complete never runs (missing "Setting newly assigned
partitions"), causing the consumer to spin without an assignment.

  • _initiate_join_group: log create vs reuse path + future state
  • join_group: log future.is_done/exception/identity after the poll,
    succeeded() branch, and entry/exit of _on_join_complete
  • ConsumerCoordinator.need_rejoin: log which branch is taken with
    the relevant snapshot/subscription state

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

Add temporary debug logging to investigate consumer rejoin flakiness
where _handle_join_success runs (logging "Enabling heartbeat thread")
but _on_join_complete never runs (missing "Setting newly assigned
partitions"), causing the consumer to spin without an assignment.

  - _initiate_join_group: log create vs reuse path + future state
  - join_group: log future.is_done/exception/identity after the poll,
    succeeded() branch, and entry/exit of _on_join_complete
  - ConsumerCoordinator.need_rejoin: log which branch is taken with
    the relevant snapshot/subscription state

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@dpkp dpkp force-pushed the claude/flakey-consumer-tests branch from a32fe2d to e51a107 Compare April 11, 2026 21:27
@dpkp dpkp merged commit 337af68 into master Apr 11, 2026
19 checks passed
@dpkp dpkp deleted the claude/flakey-consumer-tests branch April 11, 2026 22:08
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