Skip to content

Backport #12137 to 20.5: Remove harmful code from "geoDistance"#12247

Merged
alexey-milovidov merged 735 commits into20.5from
backport/20.5/12137
Jul 7, 2020
Merged

Backport #12137 to 20.5: Remove harmful code from "geoDistance"#12247
alexey-milovidov merged 735 commits into20.5from
backport/20.5/12137

Conversation

@robot-clickhouse
Copy link
Member

Original pull-request #12137
Cherry-pick pull-request #12220

This pull-request is a last step of an automated backporting.
Treat it as a standard pull-request: look at the checks and resolve conflicts.
Merge it only if you intend to backport changes to the target branch, otherwise just close it.

alexey-milovidov and others added 30 commits June 28, 2020 01:37
xeus-clickhouse is a Jupyter kernal for ClickHouse
client exit with unrecognized arguments
Revert "[experiment] maybe fix warnings in integration tests"
alexey-milovidov and others added 26 commits July 4, 2020 01:52
target_compile_definitions may only set INTERFACE properties on IMPORTED targets
* Add unbundled mode flag

* Move skip lists to clickhouse-test

* More verbose message about skip

* Make skip-list optional

* Update clickhouse-test
By default IMPORTED target has a scope in the directory in which it is created
and below. This leads to the following issues when building UNBUNDLED:

Target "clickhouse" links to target "Poco::Data::ODBC" but the target was not
found.  Perhaps a find_package() call is missing for an IMPORTED target, or an
ALIAS target is missing?
* simplify reading in order of sorting key

* add perf test for reading many parts

* Revert "simplify reading in order of sorting key"

This reverts commit 7267d7c.

* add threshold for preliminary merge for reading in order

* better threshold

* limit threads in test
Added a showcase of minimal Docker image
Minor cmake fixes for UNBUNDLED build
Fix Parquet for LowCardinality
Fix result_rows and result_bytes metrics for selects.
Add missed <atomic> include in ProxyListConfiguration
Remove harmful code from "geoDistance"
…2e9e508ac06a1f195d2b405ce38de

Cherry pick #12137 to 20.5: Remove harmful code from "geoDistance"
@robot-clickhouse robot-clickhouse added the pr-backport Changes, backported to release branch. Do not use manually - automated use only! label Jul 7, 2020
@alexey-milovidov alexey-milovidov merged commit 5f8be79 into 20.5 Jul 7, 2020
@alexey-milovidov alexey-milovidov deleted the backport/20.5/12137 branch July 7, 2020 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backport Changes, backported to release branch. Do not use manually - automated use only!

Projects

None yet

Development

Successfully merging this pull request may close these issues.