Skip to content
This repository has been archived by the owner on Sep 16, 2021. It is now read-only.

Consider renaming LightblueRequester to AsyncLightblueClient; move to lightblue client library #57

Open
alechenninger opened this issue Jul 12, 2016 · 2 comments

Comments

@alechenninger
Copy link
Contributor

The interface and the bulk implementation are general purpose enough that it should probably be shared with all client library users.

@dcrissman
Copy link
Member

I am not sure that I understand the rename suggestion as the LightblueRequester is not a descendent of LightblueClient, which I think would be confusing.

@alechenninger
Copy link
Contributor Author

alechenninger commented Nov 12, 2016

Not a descendant, but it's still a lightblue client logically speaking. There are many libraries that have a [Something]Client and an Async[Something]Client for the same service (for example Apache Thrift or gRPC code gen, I've seen it elsewhere too). They are just two different interface styles–sync vs async–to the same thing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants