search

nicholassm / disruptor-rs

Low latency inter-thread communication library in Rust inspired by the LMAX Disruptor.
MIT License
618 stars 18 forks source link

How to move Producer? #3

Closed ghost closed 4 months ago

ghost commented 4 months ago

Hi,

I would like to move the producer to a function, but the producer crate is private, how can I specify the type for

MultiProducer<EventDisrupt, SingleConsumerBarrier>

Is it even possible ?

nicholassm commented 4 months ago

Hi,

That's absolutely not the intention! If the privacy is too restrictive, I'll loosen it.

Stay tuned, I'm traveling the next few days so won't be able to do coding while I'm away.

Kind regards Nicholas

ons. 24. apr. 2024 22.18 skrev aasimovv @.***>:

Hi,

I would like to move the producer to a function, but the producer crate is private, how can I specify the type for

MultiProducer<EventDisrupt, SingleConsumerBarrier>

Is it even possible ?

— Reply to this email directly, view it on GitHub https://github.com/nicholassm/disruptor-rs/issues/3, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABJ7D2AJOBUT3HNL463PKTY7AHQXAVCNFSM6AAAAABGXS5S4WVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3DEMJRGM3TIOI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

ghost commented 4 months ago

Hey, I've made a PR for this: https://github.com/nicholassm/disruptor-rs/pull/5

nicholassm commented 4 months ago

Hi Aasimovv,

Many thanks for the PR. I'll look at it when I'm back at my laptop.

Kind regards Nicholas

fre. 26. apr. 2024 22.36 skrev aasimovv @.***>:

Hey, I've made a PR for this: #5 https://github.com/nicholassm/disruptor-rs/pull/5

— Reply to this email directly, view it on GitHub https://github.com/nicholassm/disruptor-rs/issues/3#issuecomment-2080084163, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABJ7D45TFMEITRKVQILZUTY7K3EDAVCNFSM6AAAAABGXS5S4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBQGA4DIMJWGM . You are receiving this because you commented.Message ID: @.***>

nicholassm commented 4 months ago

You can use version 1.1.0 now with the re-exports. Thanks.