Commit Graph

147068 Commits

Author SHA1 Message Date
János Benjamin Antal
a177bb6307 Really fix the bug 2024-07-15 15:15:36 +00:00
János Benjamin Antal
38027abbec Fix bug in numbers when both limit and offset is used but the index cannot be used 2024-07-15 15:15:36 +00:00
Max K
8340cec8b7
Merge pull request #66524 from ClickHouse/ci_fix_await
CI: Multiple fixes for handling killed jobs
2024-07-15 13:58:10 +00:00
Nikita Fomichev
f944eb1436
Merge pull request #66522 from ClickHouse/rename-bad-lognames
Tests: rename bad log names
2024-07-15 13:36:26 +00:00
Max K
43b3a133e9 add prints to check_mergeable_state 2024-07-15 15:17:16 +02:00
Max K
f7dfa21bff style fixes 2024-07-15 14:39:29 +02:00
Max K
1e6edbd4ae ci buddy message format updates 2024-07-15 14:14:44 +02:00
Max K
31fdfbc737 add fetching exit code if job was killed 2024-07-15 14:10:31 +02:00
Max K
d768251f99 fix handling of skipped build 2024-07-15 13:43:07 +02:00
Max K
6f036fad59 CI: Fix ci await function 2024-07-15 13:35:20 +02:00
max-vostrikov
8a5c66eadd
Merge pull request #66432 from ClickHouse/bug_66333
Add a test for #66333
2024-07-15 11:17:11 +00:00
Nikita Fomichev
2a0253d1e2 Tests: rename bad log names 2024-07-15 13:14:29 +02:00
Raúl Marín
e4f348acc7
Merge pull request #66404 from Algunenano/mergetree_private
Untangle setting headers
2024-07-15 08:49:05 +00:00
Max K
a8a04f9e22
Merge pull request #66512 from ClickHouse/ci_fix_builds_2_skip_in_prs
CI: Fix for skipping Builds_2 in PRs' CI
2024-07-15 08:29:19 +00:00
Max K
f311612e91 check filter_out_not_affected_jobs in unit test 2024-07-15 09:59:23 +02:00
Max K
7ef89f169b CI: Fix for skipping Builds_2 in PRs' CI 2024-07-15 09:26:07 +02:00
Robert Schulze
7276ad2888
Merge pull request #66163 from Blargian/docs_partitionID
Add alias `partitionID` for function `partitionId` and add documentation
2024-07-15 06:58:24 +00:00
Robert Schulze
cd719bf8fc
Make the alias case-sensitive 2024-07-15 08:57:11 +02:00
Antonio Andelic
7d62a19f4c
Merge pull request #66439 from azat/jemalloc-clock-assertion
[RFC] Fix jemalloc assertion due to non-monotonic CLOCK_MONOTONIC_COARSE
2024-07-15 06:45:29 +00:00
Alexey Milovidov
89f8c577e4 Do not fix anything 2024-07-15 05:34:14 +02:00
Yakov Olkhovskiy
77396d00d9
Merge pull request #65267 from wudidapaopao/fix_unexpected_cte_projection_name
Fix unexpected projection name when query with CTE
2024-07-15 02:14:42 +00:00
Alexey Milovidov
697f4eae97
Merge pull request #66480 from Blargian/patch-5
Update installation options page
2024-07-14 19:48:55 +00:00
Alexey Milovidov
0e671f72d9
Merge pull request #63643 from ClickHouse/everything-should-work-with-analyzer
Everything should work with Analyzer
2024-07-14 21:52:59 +02:00
Alexey Milovidov
deead28fee
Merge pull request #66499 from ClickHouse/add-one-more-revision-to-ignore
Add one more revision to ignore
2024-07-14 19:26:17 +00:00
Alexey Milovidov
1c24085d5b
Merge pull request #66483 from Blargian/patch-6
Update developer-instruction.md
2024-07-14 18:58:49 +00:00
Alexey Milovidov
fd44c533cf Add one more revision to ignore 2024-07-14 20:57:02 +02:00
Alexey Milovidov
6f3d5704d8
Merge pull request #66489 from rschu1ze/openssl-tsan-follow-up
OpenSSL: Minor follow-up to #66064
2024-07-14 18:54:07 +00:00
alesapin
6a322b5357
Merge pull request #66249 from nauu/rename-to-avoid-ambiguity
Rename FilesystemCacheFailToReserveSpaceBecauseOfLockContention to FilesystemCacheFailToReserveSpaceBecauseOfCacheResize
2024-07-14 14:04:17 +00:00
Nikita Mikhaylov
451e99dcbc
Merge pull request #66482 from ClickHouse/add-test-37557
Add a test for #37557
2024-07-14 13:29:14 +00:00
Robert Schulze
3c39c4f514
Merge remote-tracking branch 'ClickHouse/master' into docs_partitionID 2024-07-14 12:40:05 +00:00
Max K
c889205d52
Merge pull request #66490 from ClickHouse/ci_fix_job_filter_prs
CI: Fix for job filtering in PRs
2024-07-14 12:37:00 +00:00
Max K
9dabf205e6 CI: Fix for await to always await for builds 2024-07-14 14:01:10 +02:00
Max K
f374cdfe69 CI: Fix for job filtering in PRs 2024-07-14 13:53:20 +02:00
Robert Schulze
ad86e3060a
OpenSSL: Follow-up to #66064
More commits were added to the upstream fix after #66064 has been merged
in ClickHouse. This PR adds these commits to keep things in-sync.
2024-07-14 11:48:13 +00:00
Max K
cff67cc0ee
Merge pull request #66488 from ClickHouse/ci_gh_retry
CI: Add retry for GH set_status_comment call
2024-07-14 11:06:24 +00:00
Robert Schulze
0b7ff4c3ee
Merge pull request #63186 from cucumber95/change_date
Add `changeDate*` functions
2024-07-14 10:46:56 +00:00
Max K
379706d2d5 fix for job filtering 2024-07-14 12:08:53 +02:00
Max K
80c8511004 CI: Add retry for GH set_comment_status call 2024-07-14 11:39:31 +02:00
Azat Khuzhin
669ce7cd40 Enable jemalloc heap profiler by default for debug builds as well
Since the issue with jemalloc assertion is clear, let's revert that
workaround

Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2024-07-14 08:21:13 +02:00
Azat Khuzhin
22611781b0 Fix jemalloc assertion due to non-monotonic CLOCK_MONOTONIC_COARSE
Recently one tricky assertion of jemalloc had been discovered [1]:

    Failed assertion: "nstime_compare(&decay->epoch, new_time) <= 0"

  [1]: https://github.com/ClickHouse/ClickHouse/issues/66193

And as it turns out it is really possible for CLOCK_MONOTONIC_COARSE to
go backwards, in a nutshell it can be done with ADJ_FREQUENCY, you can
find example here [2]. And I can't trigger this issue for non-coarse
clocks.

  [2]: https://gist.github.com/azat/7ea7f50ed75591b1af2d675a240ea94c?permalink_comment_id=5119222#gistcomment-5119222

But, jemalloc do not call clock_gettime() that frequently (I've verified
it), so it can use non-coarse version - CLOCK_MONOTONIC

I've also measured the latency of CLOCK_MONOTONIC and
CLOCK_MONOTONIC_COARSE, and it is 20ns vs 4ns per call [3], so make this
change affect performance you need really frequently calls of
clock_gettime.

  [3]: https://gist.github.com/azat/622fa1f9a5d8e7d546ee9d294501961d?permalink_comment_id=5119245#gistcomment-5119245

Interesting, that this bug started to appears only after jemalloc heap
profiler had been enabled by default [4], no clue why (I would believe
more in a more frequent calls to clock_adjtime(ADJ_FREQUENCY), but I
can't verify this)

  [4]: https://github.com/ClickHouse/ClickHouse/pull/65702

To be continued...

Fixes: https://github.com/ClickHouse/ClickHouse/issues/66193
Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2024-07-14 08:21:13 +02:00
Shaun Struwig
5daac46d8c
Merge branch 'ClickHouse:master' into docs_partitionID 2024-07-14 05:34:01 +02:00
Alexey Milovidov
045a7f4b8f Merge branch 'master' of github.com:ClickHouse/ClickHouse into add-test-37557 2024-07-14 01:10:39 +02:00
Alexey Milovidov
4237912d4e Merge branch 'master' into add-test-37557 2024-07-14 01:10:29 +02:00
Shaun Struwig
a90e10c438 add alias partitionI, update tests, and docs 2024-07-13 22:38:28 +00:00
Alexey Milovidov
860db03d13
Merge pull request #66460 from ClickHouse/tavplubix-patch-10
Update 00992_system_parts_race_condition_zookeeper_long.sh
2024-07-13 22:21:00 +00:00
Robert Schulze
dfb831f861
Try to fix an msan issue 2024-07-13 22:07:46 +00:00
Alexey Milovidov
a8219d3848
Merge pull request #66233 from arthurpassos/remove_concurrent_auth_storm
Remove host regexp concurrent integration test
2024-07-13 21:37:58 +00:00
Alexey Milovidov
e3a2a2e487
Merge pull request #66353 from ClickHouse/collect-minio-logs
Collect logs from `minio` in stateless and statefull tests
2024-07-13 21:34:50 +00:00
Alexey Milovidov
6ba7765b81
Update 03203_multiif_and_where_2_conditions_old_analyzer_bug.sql 2024-07-13 23:33:29 +02:00
Alexey Milovidov
e71f0fe385
Merge pull request #66375 from azat/dist-fix-log
Fix "Sending a batch of X files to Y (0.00 rows, 0.00 B bytes)." in case of batch restoring
2024-07-13 21:32:21 +00:00