Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Postponed] Replace IDocumentClientEx with proper SDK interface -- when it comes online #15

Open
kadukf opened this issue Dec 20, 2017 · 4 comments

Comments

@kadukf
Copy link
Contributor

kadukf commented Dec 20, 2017

Extend DocDB SDK so that IXyzEx interfaces + their implementations are not needed.

@ealsur
Copy link
Member

ealsur commented Dec 26, 2017

The current IDocumentClient does not yet expose CreateDocumentChangeFeedQuery and ReadPartitionKeyRangeFeedAsync.

We need those two methods in order to replace the actual interfaces completely.

@mkolt
Copy link
Contributor

mkolt commented Jan 26, 2018

Do we need this? SDK Ixxx interfaces would probably always be behind. IDocumentClientEx + others are not bad.

@mkolt mkolt changed the title Replace IDocumentClientEx with proper SDK interface [P2] Replace IDocumentClientEx with proper SDK interface -- when it comes online Feb 2, 2018
@mkolt
Copy link
Contributor

mkolt commented Feb 10, 2018

SDK changes are not going to be in nearest future. Will have to postpone this until then.

@mkolt mkolt changed the title [P2] Replace IDocumentClientEx with proper SDK interface -- when it comes online [Postponed] Replace IDocumentClientEx with proper SDK interface -- when it comes online Feb 10, 2018
@mkolt
Copy link
Contributor

mkolt commented Nov 19, 2018

We should close this.

IChangeFeedDocumentClient is self-consistent interface that outlines what is needed to CFP and has worked well. IDocumentClient with its 200+ methods it too much.

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

No branches or pull requests

3 participants