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

fix: remove actor system for transactions, refactor iTX and tracing #5001

Draft
wants to merge 26 commits into
base: main
Choose a base branch
from

Conversation

aqrln
Copy link
Member

@aqrln aqrln commented Sep 21, 2024

@aqrln aqrln added this to the 5.20.0 milestone Sep 21, 2024
@aqrln aqrln self-assigned this Sep 21, 2024
@aqrln aqrln changed the title fix: remove actor system for transactions fix: remove actor system for transactions, refactor iTX and tracing Sep 21, 2024
Copy link
Contributor

github-actions bot commented Sep 21, 2024

WASM Query Engine file Size

Engine This PR Base branch Diff
Postgres 2.046MiB 2.063MiB -17.666KiB
Postgres (gzip) 817.387KiB 824.209KiB -6.822KiB
Mysql 2.019MiB 2.034MiB -15.495KiB
Mysql (gzip) 805.658KiB 812.033KiB -6.375KiB
Sqlite 1.913MiB 1.929MiB -16.638KiB
Sqlite (gzip) 764.973KiB 770.620KiB -5.647KiB

Copy link

codspeed-hq bot commented Sep 21, 2024

CodSpeed Performance Report

Merging #5001 will not alter performance

Comparing integration/itx-refactor (d9c03d9) with main (031f4d3)

Summary

✅ 11 untouched benchmarks

@jkomyno jkomyno assigned jkomyno and unassigned jkomyno Sep 23, 2024
@jkomyno
Copy link
Contributor

jkomyno commented Sep 23, 2024

I'm waiting for #5003 before being able to release a new integration version.

@aqrln aqrln force-pushed the integration/itx-refactor branch 4 times, most recently from 21703fd to 0014c6b Compare September 26, 2024 14:47
@aqrln aqrln modified the milestones: 5.20.0, 5.21.0 Sep 26, 2024
@aqrln aqrln force-pushed the integration/itx-refactor branch 2 times, most recently from d9c03d9 to ff4016d Compare October 10, 2024 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment