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

Prevent logging data from being affected by transactions. #392

Merged
merged 7 commits into from
Mar 7, 2022

Commits on Feb 27, 2022

  1. Core implementation without config handling.

    * Added class SqlConnectionStringBuilderWrapper to add Enlist= parameter. Used in in SqlConnectionFactory.
    * Added new sink option PreventEnlistInTransaction in code and documented it in README.
    * Tests
    ckadluba committed Feb 27, 2022
    Configuration menu
    Copy the full SHA
    2dc82b1 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    542690b View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    e0844a2 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    8c3ba3c View commit details
    Browse the repository at this point in the history

Commits on Mar 4, 2022

  1. Changes after review from @jonorossi.

    * Renamed PreventEnlistInTransactions to EnlistInTransaction and changed default to false.
    * Fixed typos in README.
    ckadluba committed Mar 4, 2022
    Configuration menu
    Copy the full SHA
    54a3962 View commit details
    Browse the repository at this point in the history
  2. Execute test LogsAreNotAffectedByTransactionsByDefault in a separate …

    …run from all other tests to avoid side effects with other DB tests. Ugly but the only feasible way to get this stable.
    ckadluba committed Mar 4, 2022
    Configuration menu
    Copy the full SHA
    46a8358 View commit details
    Browse the repository at this point in the history

Commits on Mar 7, 2022

  1. Ensured stability of DB tests.

    * Increased connection timeout when deleting or creating DB in DB tests (enables stable parallel execution of all DB tests including the transaction test).
    * Let all tests execute in one run again. No more separation of transaction DB test necessary.
    ckadluba committed Mar 7, 2022
    Configuration menu
    Copy the full SHA
    11cb6a3 View commit details
    Browse the repository at this point in the history