blazingmq icon indicating copy to clipboard operation
blazingmq copied to clipboard
verified publisher icon bloomberg

BlazingMQ C SDK Initial Pull Request

Open Simon-Sandrew opened this issue 2 years ago • 7 comments

Initial PR for the C SDK. Requesting review on everything in the z_bmq folder.

Simon-Sandrew avatar Dec 08 '23 20:12 Simon-Sandrew

@Simon-Sandrew Thanks for the PR! Two quick things:

  1. Could you run clang-format over your changes? The CI Formatting Check check is failing.
  2. Could you sign off your commits following https://github.com/bloomberg/blazingmq/blob/main/CONTRIBUTING.md#dco-check? The CI DCO check is failing.

kaikulimu avatar Dec 14 '23 00:12 kaikulimu

@hallfox Would you like to take a look at whether the basic code layout and directory structure make sense? And then I can take a more-depth look.

kaikulimu avatar Dec 14 '23 01:12 kaikulimu

Addresses #114

hallfox avatar Dec 14 '23 16:12 hallfox

Out of interest, is there an ETA for getting an initial C SDK merged? I’m considering experimenting with Rust bindings and a C / FFI interface would be awesome.

marvin-hansen avatar Mar 22 '24 13:03 marvin-hansen

Out of interest, is there an ETA for getting an initial C SDK merged? I’m considering experimenting with Rust bindings and a C / FFI interface would be awesome.

Hi Marvin, thanks for the interest! The ETA to merge this PR is around a month. Also, we have been experimenting with Rust bindings over this C API as well. We can find some time to discuss our ideas.

kaikulimu avatar Mar 22 '24 14:03 kaikulimu

Excellent. Around a month sounds good to me.

Background is, I just started a multi exchange data streaming project in Rust and, obviously, BlazingMQ would be an ideal candidate as low latency messaging middleware.

Let’s discuss and exchange ideas. Please send me a message using the contact form below and we find sone time.

https://emet-labs.com/contact/

On Fri, Mar 22, 2024 at 10:36 PM Yuan Jing Vincent Yan < @.***> wrote:

Out of interest, is there an ETA for getting an initial C SDK merged? I’m considering experimenting with Rust bindings and a C / FFI interface would be awesome.

Hi Marvin, thanks for the interest! The ETA to merge this PR is around a month. Also, we have been experimenting with Rust bindings over this C API as well. We can find some time to discuss our ideas.

— Reply to this email directly, view it on GitHub https://github.com/bloomberg/blazingmq/pull/169#issuecomment-2015240990, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFYR7XHNUQLY2T45IDU5HRDYZQ6X7AVCNFSM6AAAAABANEW2DKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJVGI2DAOJZGA . You are receiving this because you commented.Message ID: @.***>

marvin-hansen avatar Mar 22 '24 15:03 marvin-hansen

@pniedzielski Adding you as a reviewer

kaikulimu avatar Apr 17 '24 16:04 kaikulimu