GitHub user haze518 added a comment to the discussion: Sans-IO iggy-client and 
runtime abstraction in the Iggy SDK

> I'm a big fan of this design, especially considering that we could introduce 
> both, synchronous clients and no_std (hopefully, at some point in the 
> future). Speaking of the mentioned client changes, do you have something 
> specific in mind already (e.g., changing the existing traits), or is it 
> mostly about resolving the initial client transport/protocol?

Regarding client changes, I do plan to eventually replace ClientWrapper with a 
generic type alias. I don’t expect this to significantly affect the public API, 
 apart from possibly adding a from_connection_string method/trait like [here in 
@vax-r's 
PR](https://github.com/apache/iggy/pull/1912/files#diff-5e5cc5bb45100fcc5329b59de905e8ecadf13c96952a5d157904fe49d2454b0cR26).
 The existing trait methods should remain mostly the same (with synchronous 
counterparts added). After talking with @marvin-hansen, I think this can 
probably be introduced later, after the synchronous client is in place and when 
we replace the old async client

GitHub link: 
https://github.com/apache/iggy/discussions/2132#discussioncomment-14250913

----
This is an automatically sent email for [email protected].
To unsubscribe, please send an email to: [email protected]

Reply via email to