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

[CF-557] Changelog updates should wait for transaction commit #137

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

[CF-557] Changelog updates should wait for transaction commit #137

wants to merge 16 commits into from

Commits on Jan 11, 2024

  1. [CF-557] Changelog updates should wait for transaction commit

    @erikdw
    erikdw committed Jan 11, 2024
    Configuration menu
    Copy the full SHA
    b8c6173 View commit details
    Browse the repository at this point in the history

  2. add back newline that got autoremoved somehow

    @erikdw
    erikdw committed Jan 11, 2024
    Configuration menu
    Copy the full SHA
    8574a13 View commit details
    Browse the repository at this point in the history

  3. pin alpine version to avoid musl problems 

    See: mattn/go-sqlite3#1164
    @erikdw
    erikdw committed Jan 11, 2024
    Configuration menu
    Copy the full SHA
    5c5b5d8 View commit details
    Browse the repository at this point in the history

Commits on Jan 12, 2024

  1. refactor to make code easier to read, and add another test case

    @erikdw
    erikdw committed Jan 12, 2024
    Configuration menu
    Copy the full SHA
    4221eb9 View commit details
    Browse the repository at this point in the history

  2. add ⚠️ to comment

    @erikdw
    erikdw committed Jan 12, 2024
    Configuration menu
    Copy the full SHA
    e8a2d85 View commit details
    Browse the repository at this point in the history

  3. changelog: propagate change type

    @erikdw
    erikdw committed Jan 12, 2024
    Configuration menu
    Copy the full SHA
    209e26c View commit details
    Browse the repository at this point in the history

  4. add another stop for the propagation trainline

    @erikdw
    erikdw committed Jan 12, 2024
    Configuration menu
    Copy the full SHA
    73d259a View commit details
    Browse the repository at this point in the history

  5. update comment

    @erikdw
    erikdw committed Jan 12, 2024
    Configuration menu
    Copy the full SHA
    6b11bd8 View commit details
    Browse the repository at this point in the history

  6. rename ChangeType to ChangeOp, defer mapping of SQLite Op until we ar… 

    …e writing out the changes, and include the ledger sequence in the changelog
    @erikdw
    erikdw committed Jan 12, 2024
    Configuration menu
    Copy the full SHA
    690d906 View commit details
    Browse the repository at this point in the history

Commits on Jan 15, 2024

  1. fix transaction tracking by simplyifying code to just allocate new sl… 

    …ices, propagate ledger sequence for each statement by storing it into the SQLiteWatchChange, also add a boolean for whether the changelog entry is from a transaction or not, and add another test case
    @erikdw
    erikdw committed Jan 15, 2024
    Configuration menu
    Copy the full SHA
    9679b40 View commit details
    Browse the repository at this point in the history

Commits on Jan 16, 2024

  1. undo Op propagation

    @erikdw
    erikdw committed Jan 16, 2024
    Configuration menu
    Copy the full SHA
    5e21030 View commit details
    Browse the repository at this point in the history

  2. add test case for two ledger transactions

    @erikdw
    erikdw committed Jan 16, 2024
    Configuration menu
    Copy the full SHA
    4dc0a35 View commit details
    Browse the repository at this point in the history

  3. add warning to ldb_writer_with_changelog.go and remove debug printouts

    @erikdw
    erikdw committed Jan 16, 2024
    Configuration menu
    Copy the full SHA
    4d4de9b View commit details
    Browse the repository at this point in the history

  4. adjust tests to account for changelog changes

    @erikdw
    erikdw committed Jan 16, 2024
    Configuration menu
    Copy the full SHA
    4479f75 View commit details
    Browse the repository at this point in the history

  5. fix incorrect copy-pasta comment

    @erikdw
    erikdw committed Jan 16, 2024
    Configuration menu
    Copy the full SHA
    a616e62 View commit details
    Browse the repository at this point in the history

  6. make API for ending transaction more clear and less subtle (remove po… 

    …ssibly premature optimization of updating passed-in slice; also remove metric for ldb_changes_accumulated in favor of just using the ldb_changes_written one
    @erikdw
    erikdw committed Jan 16, 2024
    Configuration menu
    Copy the full SHA
    7f3a202 View commit details
    Browse the repository at this point in the history