8000 Update usage-dump payload by shimonewman · Pull Request #9085 · appwrite/appwrite · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update usage-dump payload #9085

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 2 commits into from
Dec 10, 2024
Merged

Update usage-dump payload #9085

merged 2 commits into from
Dec 10, 2024

Conversation

shimonewman
Copy link
Contributor

Update project payload attributes in order to reduce the size of the payload being stored on REDIS.
Update the number of keys per dump

Copy link
github-actions bot commented Dec 9, 2024

Security Scan Results for PR

Docker Image Scan Results

Package Version Vulnerability Severity
pyc 3.12.7-r0 CVE-2024-12254 HIGH
python3 3.12.7-r0 CVE-2024-12254 HIGH
python3-pyc 3.12.7-r0 CVE-2024-12254 HIGH
python3-pycache-pyc0 3.12.7-r0 CVE-2024-12254 HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

Copy link
github-actions bot commented Dec 9, 2024

✨ Benchmark results

  • Requests per second: 1,082
  • Requests with 200 status code: 194,879
  • P99 latency: 0.168517772

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 1,082 1,625
200 194,879 292,487
P99 0.168517772 0.071488104

@christyjacob4 christyjacob4 merged commit 203e110 into 1.6.x Dec 10, 2024
62 of 63 checks passed
@christyjacob4 christyjacob4 deleted the update-usage-payload branch December 10, 2024 13:25
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.

2 participants
0