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

chore: extract sql-query-builder and query-builder crates #5124

Merged
merged 1 commit into from
Jan 15, 2025
Merged

chore: extract sql-query-builder and query-builder crates #5124

merged 1 commit into from
Jan 15, 2025

Conversation

jacek-prisma
Copy link
Contributor

@jacek-prisma jacek-prisma commented Jan 15, 2025 •
edited
Loading

Getting #5123 on the main branch

@jacek-prisma jacek-prisma requested a review from a team as a code owner January 15, 2025 10:02
@jacek-prisma jacek-prisma requested review from FGoessler and aqrln and removed request for a team January 15, 2025 10:02
@github-actions GitHub Actions
Copy link
Contributor

WASM Query Engine file Size

Engine This PR Base branch Diff
Postgres 2.105MiB 2.098MiB 7.809KiB
Postgres (gzip) 845.675KiB 843.000KiB 2.676KiB
Mysql 2.065MiB 2.059MiB 6.147KiB
Mysql (gzip) 831.065KiB 828.996KiB 2.070KiB
Sqlite 1.978MiB 1.974MiB 4.031KiB
Sqlite (gzip) 793.975KiB 793.584KiB 401.000B

@codspeed-hq CodSpeed HQ
Copy link

codspeed-hq bot commented Jan 15, 2025

CodSpeed Performance Report

Merging #5124 will not alter performance

Comparing chore/extract-sql-query-builder-main (47364ea) with main (5700ed6)

Summary

✅ 11 untouched benchmarks

@aqrln aqrln merged commit 1635fdd into main Jan 15, 2025
366 of 367 checks passed
@aqrln aqrln deleted the chore/extract-sql-query-builder-main branch January 15, 2025 12:18
@aqrln aqrln mentioned this pull request Jan 17, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aqrln aqrln aqrln approved these changes

@FGoessler FGoessler Awaiting requested review from FGoessler FGoessler is a code owner automatically assigned from prisma/orm-rust

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
@jacek-prisma @aqrln