8000 Feat: Bulk Create Documents by PineappleIOnic · Pull Request #8968 · appwrite/appwrite · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
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.

Sign up for GitHub

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

Jump to bottom

Feat: Bulk Create Documents #8968

Merged
merged 26 commits into from
Dec 4, 2024
Merged

Feat: Bulk Create Documents #8968

merged 26 commits into from
Dec 4, 2024

Conversation

PineappleIOnic
Copy link
Contributor

What does this PR do?

This PR modifies the existing create document route to allow multiple documents to be created in one API call by adding a 'documents' param.

Test Plan

Old document create tests will stay in place to ensure we don't break functionality, and new create documents tests will be added to ensure added functionality works as expected around the various special DB features such as Relationships and Permissions.

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?

@github-actions GitHub Actions
Copy link
github-actions bot commented Nov 13, 2024 •
edited
Loading

Security Scan Results for PR

Docker Image Scan Results

8000
Package Version Vulnerability Severity
libecpg 16.3-r0 CVE-2024-10979 HIGH
libecpg 16.3-r0 CVE-2024-7348 HIGH
libecpg-dev 16.3-r0 CVE-2024-10979 HIGH
libecpg-dev 16.3-r0 CVE-2024-7348 HIGH
libpq 16.3-r0 CVE-2024-10979 HIGH
libpq 16.3-r0 CVE-2024-7348 HIGH
libpq-dev16.3-r0 CVE-2024-10979 HIGH
libpq-dev 16.3-r0 CVE-2024-7348 HIGH
postgresql16-dev 16.3-r0 CVE-2024-10979 HIGH
postgresql16-dev 16.3-r0 CVE-2024-7348 HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

@github-actions GitHub Actions
Copy link
github-actions bot commented Nov 13, 2024 •
edited
Loading

✨ Benchmark results

  • Requests per second: 1,113
  • Requests with 200 status code: 200,307
  • P99 latency: 0.093688332

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 1,113 2,038
200 200,307 366,883
P99 0.093688332 0.075336449

abnegate
abnegate reviewed Nov 25, 2024
View reviewed changes
@abnegate abnegate merged commit 8c1f6aa into feat-bulk-operations Dec 4, 2024
25 checks passed
@abnegate abnegate deleted the feat-bulk-create branch December 4, 2024 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abnegate abnegate abnegate approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@PineappleIOnic @abnegate
0