Skip to content

Run the concurrent first produce checks on JRuby - #1083

Merged
mensfeld merged 1 commit into
masterfrom
49686-remove-s3-jruby-skips
Oct 31, 2026
Merged

mensfeld merged 1 commit into
masterfrom
49686-remove-s3-jruby-skips

Conversation

@coipond-coder

@coipond-coder coipond-coder Bot commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Redmine: 49686

Removes the two JRuby skips added in #1056 that waited for #1055 (now merged):

  • spec/lib/rdkafka/producer_spec.rb: rspec skip on the concurrent first produce example
  • spec/integrations/producer_concurrent_first_produce_spec.rb: JRuby SKIP branch when close timed out

Spec-only. No lib/ change, no CHANGELOG.

Verification

Local, broker from docker-compose.yml, 3 runs each:

  • MRI 4.0.8: spec/integrations/producer_concurrent_first_produce_spec.rb PASS 3/3 (incl. handle destruction after close); spec/lib/rdkafka/producer_spec.rb 109 examples, 0 failures, 3/3
  • JRuby 10.0.7.0: script PASS 3/3 (incl. handle destruction after close); producer_spec.rb 109 examples, 0 failures, 2 pending (the fork examples), 3/3. The formerly skipped example runs and passes.
  • RuboCop: no offenses

Performance impact

none (spec-only)

@coipond-coder
coipond-coder Bot marked this pull request as ready for review October 30, 2026 07:58
@coipond-coder
coipond-coder Bot requested a review from mensfeld October 30, 2026 07:58
@mensfeld
mensfeld merged commit 9621c36 into master Oct 31, 2026
162 checks passed
@mensfeld
mensfeld deleted the 49686-remove-s3-jruby-skips branch October 31, 2026 11:14
mensfeld added a commit to karafka/karafka-rdkafka that referenced this pull request Nov 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant