Skip to content

Conversation

mfreeman451
Copy link
Contributor

proton.1.0.11

High-performance OCaml driver for Timeplus Proton (ClickHouse native protocol)
A feature-rich OCaml driver for Timeplus Proton, supporting streaming queries, async inserts, compression (LZ4/ZSTD), TLS security, connection pooling, and rich data types including Arrays, Maps, Enums, and DateTime64.



🐫 Pull-request generated by opam-publish v2.5.1

Copy link
Member

@jmid jmid left a comment

Choose a reason for hiding this comment

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

Thanks!

CI summary:

  • 2 opam-2.0 failures
  • a 32-bit failure during @runtest (I've made a suggestion to address this)
  • a Windows failure during the install of conf-liblz4 (currently not supported on Windows)

Overall Lgtm!

Would you consider adding an x-maintenance-intent entry?
https://github.com/ocaml/opam-repository/blob/master/governance/policies/archiving.md

@mfreeman451
Copy link
Contributor Author

Thanks for the feedback once again @jmid, any advice on how the win32/liblz4 stuff? Can I avoid that test some how?

@jmid
Copy link
Member

jmid commented Sep 10, 2025

Thanks for the feedback once again @jmid, any advice on how the win32/liblz4 stuff? Can I avoid that test some how?

This is now down to just

  • two opam-2.0 failures and
  • a Windows MinGW failure during installation of conf-liblz4

As such, it is not a problem with your package, and it should not count against merging it!
There's a lot of packages that need support for MinGW and we just haven't gotten to updating them all. I might take a stab at conf-liblz4 one of the next days.
If you wait, this PR can be rebased afterwards. Alternatively, your package should start working once conf-liblz4 is fixed (unless other issues show up). As such, I'm a bit hesitant about disabling proton on Windows up front... 🤷

@mfreeman451 mfreeman451 deleted the opam-publish-proton.1.0.11 branch September 11, 2025 01:13
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.

2 participants