-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: add initial simple loadgen #162
Conversation
This commit adds an initial port of the simple load generator from the python/js SDKs. It will need some additional error handling, and we need to add a client-side max TPS flag, but should be usable for initial tuning work.
/** | ||
* Configures the Momento client to timeout if a request exceeds this limit. | ||
* Momento client default is 5 seconds. | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Try using triple-front-slash ///
for the C# docstrings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks gtk
<ItemGroup> | ||
<PackageReference Include="Microsoft.Extensions.Logging" Version="6.0.0" /> | ||
<PackageReference Include="Microsoft.Extensions.Logging.Console" Version="6.0.0" /> | ||
<PackageReference Include="Momento.Sdk" Version="0.27.1" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Current release is 0.29.0, which I'm now realizing probably affects the other examples as well. Not to mention that will be wrong as soon as we cut a new release, so you may want to set this to 0.29.1 if you plan to release for this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cool yeah i will update next time i'm in there but I don't think there are any perf-impacting changes in recent releases.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGMT for first iteration 🚢 . Why no f# version
This commit adds an initial port of the simple load generator
from the python/js SDKs. It will need some additional error
handling, and we need to add a client-side max TPS flag, but
should be usable for initial tuning work.