Skip to content
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

Add additional options to matching simulation cases #6240

Merged
merged 1 commit into from
Aug 21, 2024

Commits on Aug 20, 2024

  1. Add additional options to matching simulation cases

    Add RecordDecisionTaskStartedTime to control the delay incurred by matching calling to history.
    
    Add TaskProcessTime to control the delay in-between polls. This is meant to represent time the client spends executing the task.
    
    Add TasksBurst to allow setting the burst value on the rate limiter, defaulting to a value of 1. Currently it uses the same value as the RPS which results in spikey behavior that causes sync matching to significantly underperform.
    
    Allow ForwarderMaxOutstandingPolls and ForwarderMaxOutstandingTasks to be zero. It's interesting to see a control without any forwarding.
    
    Additionally add a few test cases to showcase different behaviors of the current matching stack. This isn't a complete set yet but each of these brings some interesting behavior.
    natemort committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    c445678 View commit details
    Browse the repository at this point in the history