- Notifications
You must be signed in to change notification settings - Fork20
Title to get attention#143
-
I'm so sorry for the title, I don't have any inspiration for the title about this topic that I would like to discuss 😄. First of all, I want to thank you for this awesome library. It's a real enjoyment to use, really easy to understand and extend. I'm the maintainer of the PHP extension forScyllaDB/Apache Cassandra. For a long time, my buddy@danielhe4rt and I wanted to rewrite the whole extension in Rust, for obvious reasons. It's an old codebase and contains some issues that aren't resolvable with the current state of the codebase (memory leaks, a mixture of C and C++, etc.). The rewrite has started onV2 Branch since then, I found some missing parts needed for the full rewrite (e.g. Access to Zend Persistent Resources, Smart String API), so I forked the repo and made these changes needed. In the spirit of OpenSource, I wanted to announce these changes to you and if they align with your vision of the project we can merge them, if not, free feel to cherry-pick the stuff you like. Some changes are breaking (like replacing symtable API with str_* functions and dropping support for PHP < 8.1). Changes can be foundHERE. |
BetaWas this translation helpful?Give feedback.
All reactions
👍 2❤️ 1
Replies: 2 comments
-
Welcome to fork, I will continue to monitor your modifications. But one thing that is inconsistent is that, I need to support |
BetaWas this translation helpful?Give feedback.
All reactions
👍 1
-
@jmjoy I agree with your point of view that many systems are PHP 7.3 and they can never be upgraded, but they also hope to improve performance. rust is the only solution |
BetaWas this translation helpful?Give feedback.
All reactions
❤️ 1
This discussion was converted from issue #142 on December 11, 2023 09:10.