search

kylesonaty / Stellar

A simple .net client for Azure CosmosDB
MIT License
3 stars 1 forks source link

Improve SQL Query method in DocumentsManager #6

Open kylesonaty opened 6 years ago

kylesonaty commented 6 years ago

Currently we just append sql for the type at the end. Lets actually handle other cases like order by.

sql = sql + " and " + result + "._type = \"" + typeof(T).FullName + "\"";
Gonkers commented 6 years ago

21bc350 isn't the best solution however it is a big improvement. It's still fragile in edge cases.

Gonkers commented 6 years ago

Also we should probably split that out into an ISqlParser or something to make it easier for unit tests and in case we find a better solution.