Skip to content

Ignore DNS errors when checking if dictionary source is local#36463

Merged
tavplubix merged 1 commit intomasterfrom
fix_36451
Apr 22, 2022
Merged

Ignore DNS errors when checking if dictionary source is local#36463
tavplubix merged 1 commit intomasterfrom
fix_36451

Conversation

@tavplubix
Copy link
Copy Markdown
Member

Changelog category (leave one):

  • Bug Fix (user-visible misbehaviour in official stable or prestable release)

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

Server might fail to start if it cannot resolve hostname of external ClickHouse dictionary. It's fixed. Fixes #36451.

@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-bugfix Pull request with bugfix, not backported by default label Apr 20, 2022
@tavplubix
Copy link
Copy Markdown
Member Author

Integration tests (asan, actions) [2/3] - test_distributed_respect_user_timeouts/test.py::test_reconnect - #36541
Integration tests (thread, actions) [3/4] - something's wrong with docker - subprocess.TimeoutExpired: Command '['docker-compose', '--env-file', '/ClickHouse/tests/integration/test_mysql_protocol/_instances_0/.env', '-p', 'roottestmysqlprotocol', '-f', '/compose/docker_compose_mysql_php_client.yml', 'up', '--force-recreate', '-d', '--no-build']' timed out after 300 seconds
Performance Comparison (actions) [4/4] - curl: (6) Could not resolve host: s3.amazonaws.com - 10 retries with sleep 1 are not enough...
Stress test (thread, actions) - Child process was terminated by signal 11 with no stacktrace and ==636==FATAL: ThreadSanitizer: internal allocator is out of memory trying to allocate 0xd0 bytes

@tavplubix tavplubix merged commit 7f50beb into master Apr 22, 2022
@tavplubix tavplubix deleted the fix_36451 branch April 22, 2022 12:28
@tavplubix tavplubix added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label May 23, 2022
robot-clickhouse pushed a commit that referenced this pull request May 23, 2022
robot-clickhouse pushed a commit that referenced this pull request May 23, 2022
tavplubix added a commit that referenced this pull request May 24, 2022
Backport #36463 to 22.4: Ignore DNS errors when checking if dictionary source is local
tavplubix added a commit that referenced this pull request May 24, 2022
Backport #36463 to 22.3: Ignore DNS errors when checking if dictionary source is local
kaynewu added a commit to kaynewu/ClickHouse that referenced this pull request Jun 9, 2022
* Backport ClickHouse#36167 to 22.3: Fix broken aliases during parsing of special operators

* Backport git fix for /build directory

* Trigger CI

* Update version to 22.3.5.20

* Backport ClickHouse#36637 to 22.3: Fix merges of wide parts with type `Object`

* Backport ClickHouse#36487 to 22.3: Add passphrase for certificates

* Backport ClickHouse#36707 to 22.3: Fix vertical merges in wide parts

* Update version to 22.3.6.5

* Backport ClickHouse#36910 to 22.3: Fix bug in keeper which could lead to corrupted compressed logs

* Update version to 22.3.7.5

* Backport ClickHouse#36866 to 22.3: Integration tests

* Backport ClickHouse#35803 to 22.3: Fix bug in indexes of not presented columns in -WithNames formats

* Backport ClickHouse#37021 to 22.3: Fixed problem with infs in `quantileTDigest`

* Backport ClickHouse#36463 to 22.3: Ignore DNS errors when checking if dictionary source is local

* Backport ClickHouse#37443 to 22.3: Functions normalize utf8 fix

* Backport ClickHouse#37336 to 22.3: Fix clash of constant strings in aggregate function, prewhere and join

* Backport ClickHouse#37690 to 22.3: Fix segfault with mysql db + show create table + named collections

Co-authored-by: robot-clickhouse <[email protected]>
Co-authored-by: Mikhail f. Shiryaev <[email protected]>
Co-authored-by: Anton Popov <[email protected]>
Co-authored-by: Maksim Kita <[email protected]>
Co-authored-by: Alexey Milovidov <[email protected]>
Co-authored-by: alesapin <[email protected]>
Co-authored-by: Kruglov Pavel <[email protected]>
Co-authored-by: Alexander Tokmakov <[email protected]>
Co-authored-by: Vladimir C <[email protected]>
Co-authored-by: Kseniia Sumarokova <[email protected]>
@Felixoid Felixoid added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Jul 14, 2022
yokofly added a commit to timeplus-io/proton that referenced this pull request Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-bugfix Pull request with bugfix, not backported by default pr-must-backport Pull request should be backported intentionally. Use this label with great care!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ClickHouse does not start if dns entry is invalid for external ClickHouse dictionary

4 participants