Nick Chapsas

Results 52 comments of Nick Chapsas

Hey @BjornDeRijcke thanks for raising this. Can you try something out so I can pinpoint the issue? Does the same thing happen if you set the TypeNameHandling as All on...

Ah I see. I actually hate that this dynamic conversion is there in the first place. I don't know what I was thinking at the time, but I haven't found...

This has to do with the Connection string you're providing. Make sure you are using a properly formatted Cosmos DB connection string.

Hello @irriss. Can you please provide a code snippet of how you are trying to do that so I can debug locally? Thanks

I think, moving it to the `CosmosStore` constructor as it's own thing would be more appropriate. The reason is that there are two ways to initialise the `CosmosStore`. 1) Using...

Don't worry about integration tests. the Cosmos DB team has been doing some stuff with the emulator and it's failing on some agents. I will deal with that problem once...

Hey @rob thanks for that. It looks really good. A few comments before I do a full review. Can you please uncomment the commented out tests? I also can't merge...

Hello @Mortana89, No I don't think that these errors should be log-level silent (even though they are app level silent) because it is very handy to aggregate them and see...

Exceptions are not meant to indicate bugs but rather exceptional behavior and I think this specific one falls under this category. I could potentially add a flag on the AI...

Nah it shouldn't be much work. The reason for the code level logging is consistency of logs. It leads to simpler and more consistent queries.