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

Fixes in ODBC dictionary reload and ODBC bridge reachability #18278

Merged
merged 3 commits into from Dec 21, 2020

Conversation

traceon
Copy link
Contributor

@traceon traceon commented Dec 20, 2020

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

Changelog category:

  • Bug Fix

Changelog entry:

Detailed description:

  • clickhouse-odbc-bridge now binds to 127.0.0.1 by default instead of localhost, this solves a problem when server tries to connect to ::1 while the bridge is listening to 127.0.0.1 when localhost resolves to those addresses.
  • ExternalQueryBuilder instance stores const-refs to members of enclosing object, and when that object is clone()'ed and the original object is destroyed, the instance of ExternalQueryBuilder, that is stored in the new enclosing object, now refers to the destroyed members and builds queries with garbage in them, accessing the destroyed objects. Fixes now closed Incorrect columns and segfault in odbc external dictionary updates #17215.
  • Seemingly a typo in ReadWriteBufferFromHTTPBase::call(). Fixed.

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Dec 20, 2020
@alexey-milovidov alexey-milovidov self-assigned this Dec 21, 2020
@filimonov
Copy link
Contributor

I guess it resolves #14489

@alexey-milovidov
Copy link
Member

Functional stateless tests flaky check (address) — fail: 1, passed: 91

@Enmk #17846

@alexey-milovidov alexey-milovidov merged commit fa68af0 into ClickHouse:master Dec 21, 2020
@traceon traceon deleted the odbc-dict-fixes branch December 21, 2020 08:17
robot-clickhouse pushed a commit that referenced this pull request Dec 21, 2020
robot-clickhouse pushed a commit that referenced this pull request Dec 21, 2020
robot-clickhouse pushed a commit that referenced this pull request Dec 21, 2020
robot-clickhouse pushed a commit that referenced this pull request Dec 21, 2020
alexey-milovidov added a commit that referenced this pull request Dec 21, 2020
Backport #18278 to 20.10: Fixes in ODBC dictionary reload and ODBC bridge reachability
alexey-milovidov added a commit that referenced this pull request Dec 21, 2020
Backport #18278 to 20.11: Fixes in ODBC dictionary reload and ODBC bridge reachability
alexey-milovidov added a commit that referenced this pull request Dec 21, 2020
Backport #18278 to 20.12: Fixes in ODBC dictionary reload and ODBC bridge reachability
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-bugfix Pull request with bugfix, not backported by default
Projects
None yet
4 participants