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

Update dependency com.lmax:disruptor to v4 - autoclosed #1376

Closed
wants to merge 1 commit into from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Jul 11, 2024

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
com.lmax:disruptor (source) 3.4.4 -> 4.0.0 age adoption passing confidence

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 - At any time (no schedule defined), 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 Mend Renovate. View repository job log here.

@renovate renovate bot requested a review from a team July 11, 2024 16:00
@renovate renovate bot force-pushed the renovate/com.lmax-disruptor-4.x branch from 01458ec to af52f0b Compare July 11, 2024 17:40
@breedx-splk
Copy link
Contributor

@jackshirazi @SylvainJuge @JonasKunz It looks like the next major version of disruptor here only supports java 11+. Do you want to stay locked on the old version or think about leaving java 8 behind?

@JonasKunz
Copy link
Contributor

We should stay with the previous version and compatible with Java 8, otherwise this extension becomes incompatible with the autoinstrumentation agent.

@trask
Copy link
Member

trask commented Jul 18, 2024

I've submitted #1384

@renovate renovate bot force-pushed the renovate/com.lmax-disruptor-4.x branch from af52f0b to 79b17a5 Compare July 18, 2024 15:36
@renovate renovate bot changed the title Update dependency com.lmax:disruptor to v4 Update dependency com.lmax:disruptor to v4 - autoclosed Jul 18, 2024
@renovate renovate bot closed this Jul 18, 2024
@renovate renovate bot deleted the renovate/com.lmax-disruptor-4.x branch July 18, 2024 16:58
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