Attempt to fix flacky test '00910_zookeeper_test_alter_compression_codecs'

[alexey-milovidov]

Changelog category (leave one):

• Build/Testing/Packaging Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fix flacky test 00910_zookeeper_test_alter_compression_codecs.

Detailed description / Documentation draft:

Alexey Milovidov, [05.03.20 15:14]
00910_zookeeper_test_alter_compression_codecs
- странно, стал не проходить почти всё время.

Alexander Sapin, [05.03.20 15:29]
[In reply to Alexey Milovidov]
Там нужно выставить настройку, чтобы альтер синхронно делался на всех репликах . А то мы запускаем альтер и сразу идём в system.columns и срабатывает твоя защита. Видимо старый успевал, а новый иногда не успевает.

Alexey Milovidov, [05.03.20 15:32]
[In reply to Alexander Sapin]
Понял, банальный рейс в тесте. Сейчас попробую выставить.

Alexey Milovidov, [05.03.20 15:35]
@Alesapin Я не понял, какую именно настройку выставлять, сейчас запутано. Есть mutations_sync и replication_alter_partitions_sync. Вторая точно не влияет, а первая теперь влияет на ALTER столбцов?

Alexey Milovidov, [05.03.20 15:38]
@Alesapin А разве не во время запроса, а во время применения ALTER, берётся табличный RWLock? А зачем?

Иван Блинков, [05.03.20 15:43]
[In reply to Alexey Milovidov]
Саша говорит, что надо выставить replication_alter_partitions_sync=2

Alexey Milovidov, [05.03.20 15:44]
[In reply to Иван Блинков]
Какой ужас. При чём тут партиции :(

Иван Блинков, [05.03.20 15:47]
[In reply to Alexey Milovidov]
Саша говорит, что она уже так называлась раньше. Не переименовывать же.

[alexey-milovidov]

Also removed test prefix to use random database.

[alexey-milovidov]

for i in {1..100}; do ./clickhouse-test --zookeeper zookeeper_test_alter_compression_codecs; done
Ok in debug build.

[alexey-milovidov]

It did not help:

THREAD_FUZZER_CPU_TIME_PERIOD_US=1000 THREAD_FUZZER_SLEEP_PROBABILITY=0.1 THREAD_FUZZER_SLEEP_TIME_US=100000 ./clickhouse-server

00910_zookeeper_test_alter_compression_codecs:                          [ OK ]

1 tests passed. 0 tests skipped.

Running 1 stateless tests.

00910_zookeeper_test_alter_compression_codecs:                          [ FAIL ] - return code 217
Received exception from server (version 20.3.1):
Code: 473. DB::Exception: Received from localhost:9000. DB::Exception: Possible deadlock avoided. Client should retry.: While executing Columns. 


Having 1 errors! 0 tests passed. 0 tests skipped.

Running 1 stateless tests.

00910_zookeeper_test_alter_compression_codecs:                          [ FAIL ] - return code 122
Received exception from server (version 20.3.1):
Code: 122. DB::Exception: Received from localhost:9000. DB::Exception: Table columns structure in ZooKeeper is different from local table structure.

[alexey-milovidov]

The setting SET replication_alter_partitions_sync = 1; also does not help.

[alesapin]

You have to set setting to 2, because 1 is default, so in the begging of the test:
SET replication_alter_partitions_sync = 2;