Open brendar opened 3 days ago
Hello reviewers! :wave: Please follow this checklist when reviewing this Pull Request.
release notes (needs details)
label if users need to know about this change.-
), and have a clear help text.Jobs
should be named in order to mark it as required
.required
, the maintainer team must be notified._vt
tables and RPCs need to be backward compatible.vtctl
command output order should be stable and awk
-able.All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 69.49%. Comparing base (
83b37b8
) to head (9ef4340
). Report is 4 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
The original test does not actually assert that the transaction was rolled back. It was just asserting that a new transaction can't see rows written by the other open transaction, which is expected behavior at the default isolation level.
This test now fails with