8000 Add dynamic keys in builds by Meldiron · Pull Request #8492 · appwrite/appwrite · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add dynamic keys in builds #8492

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

Merged
merged 4 commits into from
Aug 12, 2024
Merged

Add dynamic keys in builds #8492

merged 4 commits into from
Aug 12, 2024

Conversation

Meldiron
Copy link
Contributor
@Meldiron Meldiron commented Aug 8, 2024

What does this PR do?

Add dynamic key env var to build worker

Test Plan

  • new tests

Related PRs and Issues

x

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

Copy link
github-actions bot commented Aug 8, 2024

✨ Benchmark results

  • Requests per second: 1,761
  • Requests with 200 status code: 316,942
  • P99 latency: 0.077962742

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 1,761 2,240
200 316,942 403,217
P99 0.077962742 0.073200764

Copy link
github-actions bot commented Aug 8, 2024

✨ Benchmark results

  • Requests per second: 1,608
  • Requests with 200 status code: 289,519
  • P99 latency: 0.081461811

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 1,608 2,046
200 289,519 368,255
P99 0.081461811 0.076598463

8000
@christyjacob4 christyjacob4 merged commit 70c933f into 1.6.x Aug 12, 2024
24 checks passed
@christyjacob4 christyjacob4 deleted the feat-dynamic-keys-build branch August 12, 2024 09:24
@stnguyen90 stnguyen90 changed the title Feat: Dynamic keys in builds Add dynamic keys in builds Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants
0