Skip to content

Add tests with DatabaseAtomic#10714

Merged
tavplubix merged 2 commits intomasterfrom
add_tests_with_database_atomic
May 7, 2020
Merged

Add tests with DatabaseAtomic#10714
tavplubix merged 2 commits intomasterfrom
add_tests_with_database_atomic

Conversation

@tavplubix
Copy link
Copy Markdown
Member

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Non-significant (changelog entry is not required)

Allows to run functional tests with Atomic database engine enabled by default.

Internal PR to run this tests in CI and remove tests with streams: https://nda.ya.ru/t/UPKN3Fpl3VxqoG

@blinkov blinkov added the pr-not-for-changelog This PR should not be mentioned in the changelog label May 6, 2020
@tavplubix tavplubix merged commit 04a24c5 into master May 7, 2020
@tavplubix tavplubix deleted the add_tests_with_database_atomic branch May 7, 2020 11:30
blinkov pushed a commit to blinkov/ClickHouse that referenced this pull request May 8, 2020
* add tests with DatabaseAtomic

* fix
tavplubix added a commit that referenced this pull request May 8, 2020
* add tests with DatabaseAtomic

* fix

(cherry picked from commit 04a24c5)
CurtizJ pushed a commit that referenced this pull request Jun 23, 2020
Fix usage of primary key wrapped into a function.

(cherry picked from commit 63c4a76)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-docs-needed pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants