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

A draft PR for Adding a basic route Get(test/route) to test the keto performance #1128

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sagkhr23
Copy link

@sagkhr23 sagkhr23 commented Nov 11, 2022

This is the first result to recreate hit the api in Jmeter using Threads_100-RampUp_2-Duration_20_loop=infinite
Threads_100-RampUp_2-Duration_20

This is the first result to recreate hit the api in Jmeter using Threads_250-RampUp_5-Duration_30_loop=infinite
Threads_250-RampUp_5-Duration_30

The time taken for just a basic router not even processing anything is this much, then the performance of the ory service for authorization is not up to the mark for using because if an authorization server takes this much time under a normal load for a basic route it is up to no good.

Related issue(s)

Checklist

  • I have read the contributing guidelines.
  • I have referenced an issue containing the design document if my change
    introduces a new feature.
  • I am following the
    contributing code guidelines.
  • I have read the security policy.
  • I confirm that this pull request does not address a security
    vulnerability. If this pull request addresses a security vulnerability, I
    confirm that I got the approval (please contact
    [email protected]) from the maintainers to push
    the changes.
  • I have added tests that prove my fix is effective or that my feature
    works.
  • I have added or changed the documentation.

Further Comments

@CLAassistant
Copy link

CLAassistant commented Nov 11, 2022

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@sagkhr23 sagkhr23 marked this pull request as draft November 11, 2022 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants