search

Azure / azure-cosmosdb-java

Java Async SDK for SQL API of Azure Cosmos DB
MIT License
54 stars 61 forks source link

Retrieving query info using new query plan retriever #111

Closed mbhaskar closed 5 years ago

mbhaskar commented 5 years ago

Removing proxy execution context Retrieving query info using new query plan retriever

This change is Reviewable

christopheranderson commented 5 years ago

Also, please create an issue/PR to track the v3 port of this change before we merge.

moderakh commented 5 years ago

Please see if you can remove the multiple addition single toObservable in the new block of code. If you could rely on Single.toFlatMap() that would be better and you can do less transformation.

Also you have test failure. other than that looks good.

mbhaskar commented 5 years ago

Also, please create an issue/PR to track the v3 port of this change before we merge.

126