Badrish Chandramouli

Results 141 comments of Badrish Chandramouli
trafficstars

Interesting, perhaps `e.AcceptSocket.NoDelay = true;` can be moved into the `try` that follows below that section, so that we do not pay for two try/catch sections.

> Interesting, perhaps `e.AcceptSocket.NoDelay = true;` can be moved into the `try` that follows below that section, so that we do not pay for two try/catch sections. If you or...

How does this feature work exactly? Is it compatible with the older connection strings? And how does it interface with unit tests?

> Somthing that might be concerning, I noticed that the constructor of `public AzureStorageNamedDeviceFactory(string serviceUri, TokenCredential credential, ILogger logger = null)` is being called multiple times with the exact same...

OK. @Meir017, if you could rebase to the latest in main, I think we can go ahead with reviewing and merging this. thank you.

> Somthing that might be concerning, I noticed that the constructor of `public AzureStorageNamedDeviceFactory(string serviceUri, TokenCredential credential, ILogger logger = null)` is being called multiple times with the exact same...

@priyanjgupta, if @Meir017 is busy then feel free to directly create a cleaned up PR on this targeting main directly, thanks.

Garnet supports pub/sub already. If you search online for examples of pub/sub in Redis, there should be a lot of hits. Let us know if you run into any bugs...

> ``` > celery -A tasks worker --pool=solo > ``` Hi @mardukbp, can you try this PR to see if Celery works after this small fix? https://github.com/microsoft/garnet/pull/604

> Using Carmine, a Redis client for Clojure that auto-generates Clojure functions from the official Redis command spec, the following example causes GarnetServer to terminate: > > ```clojure > (ns...