vax-r opened a new pull request, #1803:
URL: https://github.com/apache/iggy/pull/1803
## Summary
Previously, the implementation of ConnectionString and
ConnectionStringOptions are tightly coupled with TCP support, making it
difficult to expand the support for other protocols, e.g. QUIC, HTTP, etc. This
created unwanted dependencies on protocol-specific details.
To bring better flexibility and maintainbility, the following changes are
made
- Convert ConnectionStringOptions from struct to trait
- Leverage protocol-specific implementation to configuration of each
protocol
- Make ConnectionString focus on string manipulation without too much
concern of protocol-specific knowledge
With this refactoring, ConnectionString now serves as a foundation that
different protocols can build upon by implementing the ConnectionStringOptions
trait. The TCP implementation has been ported to this new architecture of
ConnectioinString. Different protocols will be built on top of it in the coming
patches.
## Related Issue
#1769
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
