[codex] Add experimental transaction support#1
Draft
pepicrft wants to merge 31 commits into
Draft
Conversation
* add dyalyzer to ci * add dialyzer to ci * in test * if * eh
match plausible prod plausible/analytics#5947
Update ClickHouse version in test workflow
* multipart requests * to false * update docs and interface * remove to string * custom multipart and slight refactor * merge * docs * doc title * more cleanup * send settings as params * Bump actions/checkout from 4 to 5 (plausible#270) Bumps [actions/checkout](https://github.com/actions/checkout) from 4 to 5. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v4...v5) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix version check (plausible#274) * changelog * add older ClickHouse to CI * tag 'json as string' test as json * release v0.5.5 * update deps * comment on why older version in ci * shorter cache key * fix internal type ordering in Variant (plausible#275) * fix internal type ordering in Variant * cleanup * link pr * release v0.5.6 * fewer changes * eh * eh x2 * readme * dialyzer * a few more tests * more tests * typos skip * eh * eh! * cleanup * continue * continue --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: hawkyre <hawkyre@gmail.com> Co-authored-by: Pablo Molina <50209142+hawkyre@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* rm duplicate line * improve docs
* fix negative integer parsing in Enum8 and Enum16 types * add pr to changelog
* adapt test to latest clickhouse * don't touch deps
* add coveralls * ignore support * mix deps.get * badge
pepicrft
force-pushed
the
codex/experimental-transactions
branch
from
1781793 to
0acf8c2
Compare
Coverage Report for CI Build 0Warning No base build found for commit Coverage: 91.214%Details
Uncovered Changes
Coverage RegressionsRequires a base build to compare against. How to fix this → Coverage Stats
💛 - Coveralls |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Why
This is the driver half of an experimental ClickHouse transaction spike for Ecto SQL sandbox support. The diff is based on
v0.6.1, which is the version smoke-tested together withecto_ch v0.8.3.Validation
mix test test/ch/connection_test.exs test/ch/faults_test.exs/tmp/clickhouse_sandbox_smokepassed against ClickHouse 26.1.2.11 with Keeper and experimental transactions enabled