Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): update dependency com.lmax:disruptor to v4 #3127

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

solrbot
Copy link
Collaborator

@solrbot solrbot commented Jan 24, 2025

This PR contains the following updates:

Package Type Update Change
com.lmax:disruptor (source) dependencies major 3.4.4 -> 4.0.0

Release Notes

LMAX-Exchange/disruptor (com.lmax:disruptor)

v4.0.0

Compare Source

  • Minimum Java version now 11
  • Issue #​323 - WorkerPool and WorkProcessor have been removed, no more Disruptor::handleEventsWithWorkerPool
  • Disruptor constructors using Executor have been removed. Use ThreadFactory instead.
  • Rolled up event handling extension interfaces on to EventHandler:
    • BatchStartAware
    • LifecycleAware
    • SequenceReportingEventHandler
  • FatalExceptionHandler and IgnoreExceptionHandler now use the JDK 9 Platform Logging API, i.e. System.Logger
  • Add rewind batch feature to the BatchEventProcessor
  • Add a maximum batch size argument to BatchEventProcessor
    • EventHandler::onBatchStart now gets both the batchSize as well as queueDepth (previously it had batchSize which reported queue depth)
  • Added documentation to EventPoller
  • Util::log2 throws if passed a non-positive argument
  • Deprecations
    • Deprecated ThreadHints.onSpinWait()
    • Deprecated Disruptor.handleExceptionsWith() - this had been javadoc deprecated since 2015 but not in the code
    • Removed previously deprecated methods
      • Ringbuffer.resetTo()
      • ConsumerRepository.getLastSequenceInChain()

Configuration

📅 Schedule: Branch creation - "* * * * *" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot

Copy link
Contributor

@malliaridis malliaridis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From previous review, this upgrade requires log4j to be updated first.

Therefore, blocked by #2895.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants