Badrish Chandramouli

Results 138 comments of Badrish Chandramouli
trafficstars

@njnicko - could you address the comments and push commits when you get a chance, thanks again for your PR.

Closing as outdated. We have a proper Helm chart package now thanks to @babykart. Would be great to see a cluster version contributed over that version, at some point. Thank...

We have a Helm chart package release now thanks to @babykart. Would be great to see support for configuration files and secrets added to that, as discussed in #594. Closing...

Hi @Meir017, thanks for raising this issue. It is a good point -- we need to update AzureStorageDevice to use managed-identity. Thanks for raising it. Would this be of interest...

Mostly addressed by https://github.com/microsoft/garnet/pull/614 - feel free to create PR if you have more ideas in this space.

We recently re-ran all our benchmarks for a research paper submission, and used Valkey (the latest Redis was very close to the latest Valkey in terms of benchmark results). Overall,...

Yes, repeatedly using ElementAt is a poor implementation with quadratic overhead. It should go to the first element and iterate from there. Anyone care to PR a fix?

> > Yes, repeatedly using ElementAt is a poor implementation with quadratic overhead. It should go to the first element and iterate from there. Anyone care to PR a fix?...

you need to seek to the start, then traverse until the required number of elements (e.g., 100, 200, etc.)