Commit Graph

865 Commits

Author SHA1 Message Date
Azat Khuzhin
7c01705231 Run ValidationChecker for sanitizers build as well
Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2024-02-13 10:50:45 +01:00
Dmitry Novik
1dfecd5e6c Merge remote-tracking branch 'origin' into analyzer-fix-const-fold 2024-02-09 11:57:23 +01:00
Antonio Andelic
160f1b7fd8 Fix logical optimizer with LowCardinality 2024-02-08 15:01:56 +01:00
Dmitry Novik
b8f74b5b05
Merge pull request #50894 from ClickHouse/storage-merge-aliases-analyzer
Analyzer: support aliases and distributed JOINs in StorageMerge
2024-02-08 11:22:00 +01:00
Dmitry Novik
d558728281
Merge branch 'master' into storage-merge-aliases-analyzer 2024-02-06 20:24:20 +01:00
Raúl Marín
e10e105ec1 Merge remote-tracking branch 'blessed/master' into more_sanity_checks 2024-02-06 17:42:36 +01:00
Raúl Marín
5d042dc815 Check pointer 2024-02-06 17:42:24 +01:00
vdimir
3642bbc05c
Merge pull request #59483 from ClickHouse/vdimir/analyzer/00917_multiple_joins_denny_crane
Analyzer: Fix denny_crane
2024-02-06 10:17:27 +01:00
Raúl Marín
40885d7f16 Add extra sanity check 2024-02-05 21:51:26 +01:00
vdimir
d3123438c7
Merge pull request #59382 from ClickHouse/vdimir/analyzer/fix_array_join_subcolumn
Fix ARRAY JOIN with subcolumns
2024-02-05 17:27:59 +01:00
vdimir
9b5ea7bb8a
Analyzer: Fix 00917_multiple_joins_denny_crane 2024-02-05 16:25:13 +00:00
Robert Schulze
dd484fc312
Simplify what happens in SELECT all, ... [...] ORDER BY ALL
Previously setting `enable_order_by_all` distinguished for ORDER BY ALL
whether we should sort by column 'all' (if given in the SELECT clause)
or by all columns. The actual behavior was not always intuitive.

Now, we throw unconditionally an exception which also simplifies the
handling a bit. Only an edge case is affected and if users really want
to run ORDER BY ALL on a column names 'all', they can alias it.
2024-02-02 09:53:35 +00:00
Robert Schulze
82a4fa0dfc
Smaller fixups 2024-02-01 11:58:43 +00:00
zhongyuankai
d076b13ac8 order by all doesn't work for select * 2024-02-01 18:51:54 +08:00
vdimir
a043227685
Fix ARRAY JOIN with subcolumns in analyzer 2024-02-01 10:36:31 +00:00
Dmitry Novik
0d21004218 WIP on StorageMerge 2024-01-31 13:50:15 +00:00
Dmitry Novik
722f3db738 Merge remote-tracking branch 'origin/master' into storage-merge-aliases-analyzer 2024-01-30 19:50:01 +00:00
Dmitry Novik
ff71973536 Merge remote-tracking branch 'origin/master' into analyzer-fix-const-fold 2024-01-30 18:43:47 +00:00
Dmitry Novik
5a024dfc09
Merge pull request #59117 from ClickHouse/fix-analyzer-order-by-all
Followup 57875
2024-01-30 15:54:18 +01:00
Dmitry Novik
c339a74ac3
Merge pull request #59177 from ClickHouse/fix-analyzer-cte-resolve
Analyzer: Fix CTE name clash resolution
2024-01-30 08:13:38 +01:00
flynn
4f12ca249d Fix typo 2024-01-30 07:01:07 +00:00
flynn
9858c41849 Merge branch 'master' of github.com:ClickHouse/ClickHouse into negtive-position 2024-01-30 04:03:35 +00:00
flynn
0ded580011 Fix 2024-01-30 04:03:27 +00:00
avogar
5833641fa5 Merge branch 'master' of github.com:ClickHouse/ClickHouse into variant-data-type 2024-01-26 16:54:02 +00:00
Dmitry Novik
6c5057c4f2
Merge pull request #58919 from ClickHouse/analyzer-inj-func-elemination
Analyzer: Support GROUP BY injective function elimination
2024-01-26 16:45:09 +01:00
Dmitry Novik
a9d2e13697 Handle CTEs more careafully 2024-01-26 15:20:09 +00:00
Dmitry Novik
e38face46b Fix style 2024-01-26 15:20:09 +00:00
Dmitry Novik
778f4371c2 Cleanup the code 2024-01-26 15:20:09 +00:00
Dmitry Novik
e3c07d4562 Analyzer: Fix CTE name clash resolution 2024-01-26 15:20:09 +00:00
Dmitry Novik
394e37894f Small improvement in QueryAnalysisPass 2024-01-26 15:03:35 +00:00
Nikolai Kochetov
75edbdd39c
Merge pull request #59147 from ClickHouse/vdimir/fix_agg_nothing_2
Fix AggregateFunctionNothing result type issues introducing it with different names
2024-01-26 14:33:38 +01:00
Dmitry Novik
a2d7bcbd5f
Merge pull request #58958 from ClickHouse/analyzer-insert-block
Analyzer: fix storage replacement with insertion block
2024-01-25 09:25:50 +01:00
Dmitry Novik
7826706ff9
Small fix 2024-01-24 16:09:26 +01:00
vdimir
bcfe691954
Use different names for nothing function with different result types 2024-01-24 09:55:09 +00:00
Dmitry Novik
8e0aea301e Analyzer: Add cast for ConstantNode from constant folding 2024-01-23 14:29:26 +00:00
Dmitry Novik
bef0fcb482
Fix is_order_by_all flag in QueryNode 2024-01-23 14:56:24 +01:00
Dmitry Novik
e9ea1307e0 Make optimization more general 2024-01-23 13:03:09 +00:00
Dmitry Novik
7a95eb8bd1
Review fixes 2024-01-23 11:59:41 +01:00
Dmitry Novik
6962859e43
Merge pull request #59068 from ClickHouse/check-optimization-setting
Followup to 57853
2024-01-23 11:24:31 +01:00
Dmitry Novik
f702afb72d Fix includes 2024-01-22 15:26:25 +00:00
Dmitry Novik
1abe1307db Make test to check the feature 2024-01-22 15:15:30 +00:00
Raúl Marín
1e875431db
Style 2024-01-22 14:59:53 +01:00
vdimir
0791c75315
Merge pull request #58654 from yariks5s/paste_join_aliases_requirement
Avoid alias duplicates in PASTE JOIN and fix aliases requirement
2024-01-22 14:48:13 +01:00
Dmitry Novik
3d0e915088
Followup to 57853 2024-01-22 14:44:07 +01:00
Raúl Marín
d9ab300300
Merge pull request #57853 from jiebinn/rewriteSumFunction
Rewrite the AST/Analyzer of sum(column +/- literal) function
2024-01-22 10:41:30 +01:00
Kruglov Pavel
20379d34d5
Merge branch 'master' into variant-data-type 2024-01-19 23:17:42 +01:00
Yarik Briukhovetskyi
e28fd94e1c
fix style 2024-01-19 20:19:34 +01:00
yariks5s
76b43a6aba small changes(revert) 2024-01-19 18:13:05 +00:00
Jiebin Sun
e9662d4d62 polish the code and adjust some code style 2024-01-20 01:43:06 +08:00
Dmitry Novik
d8b5623c67 Analyzer: Support GROUP BY injective function elimination 2024-01-19 14:00:12 +00:00
vdimir
b8340afddf
Merge pull request #58943 from ClickHouse/vdimir/analyzer_column_not_found
Analyzer: fix join column not found with compound identifiers
2024-01-19 10:40:03 +01:00
Jiebin Sun
0fb7d2290c
Update src/Analyzer/Passes/RewriteSumFunctionWithSumAndCountPass.cpp
Co-authored-by: Raúl Marín <git@rmr.ninja>
2024-01-19 17:01:11 +08:00
Yakov Olkhovskiy
43aaccdaa1 clang tidy 2024-01-18 19:17:05 +00:00
Yakov Olkhovskiy
b914938c4c clenup 2024-01-18 18:10:54 +00:00
Yakov Olkhovskiy
bb1058e2b2 fix storage replacement with insertion block for analyzer 2024-01-18 17:59:23 +00:00
Kruglov Pavel
5444cde408
Merge branch 'master' into variant-data-type 2024-01-18 18:31:27 +01:00
Anton Popov
d2142041cd Merge remote-tracking branch 'upstream/master' into HEAD 2024-01-18 16:03:54 +00:00
yariks5s
6ad6d6a121 fix due to review 2024-01-18 15:41:25 +00:00
vdimir
f2e73c13a9
Analyzer: fix join column not found with compound identifiers 2024-01-18 12:30:28 +00:00
Jiebin Sun
206e1c87e0 revise the comments 2024-01-18 19:07:02 +08:00
Anton Popov
94bb9a171d fix FunctionToSubcolumnsPass 2024-01-17 21:36:23 +00:00
Smita Kulkarni
d3b4dea805 Fix clang tidy build 2024-01-17 20:02:17 +01:00
Dmitry Novik
3a1c3638e8 Style fix 2024-01-17 15:04:50 +00:00
Yarik Briukhovetskyi
38914ef70c
Update QueryAnalysisPass.cpp 2024-01-17 15:48:26 +01:00
Yarik Briukhovetskyi
f05733e7b8
Update QueryAnalysisPass.cpp 2024-01-17 15:28:02 +01:00
Jiebin Sun
784cab638a rename some function_name and add more tests 2024-01-17 22:10:08 +08:00
yariks5s
66f71008b3 revert and fix 2024-01-16 15:36:43 +00:00
Smita Kulkarni
76ba60aa24 Merge branch 'master' into Support_parameterized_view_with_analyzer 2024-01-16 11:26:41 +01:00
vdimir
d2dccf9a22
Merge pull request #49703 from ClickHouse/vdimir/analyzer_join_resolve_nested
Analyzer: Fix resolving subcolumns in JOIN
2024-01-16 11:24:02 +01:00
Smita Kulkarni
605c76e66e Fix test fails 2024-01-16 11:22:27 +01:00
Alexey Milovidov
8973e7d873
Merge pull request #58833 from ClickHouse/fix-analyzer-remote-aggregation
Analyzer: Resolve GROUPING function on shards
2024-01-15 23:33:14 +01:00
yariks5s
b4f078e5cc fix bug with several joins 2024-01-15 16:49:42 +00:00
Dmitry Novik
de115c4da7 Fix for passes that use column source 2024-01-15 16:25:39 +00:00
Dmitry Novik
0f7daeffc8 Analyzer: Resolve GROUPING function on shards 2024-01-15 16:02:37 +00:00
vdimir
e9ab8e8a7d
update comment to check_nested_column_not_in_using 2024-01-15 10:58:37 +00:00
Yakov Olkhovskiy
62660e6e84
Merge pull request #58703 from ClickHouse/imp-error-message
Add identifier name to the error INVALID_IDENTIFIER
2024-01-14 07:46:33 -05:00
Yakov Olkhovskiy
ef880657ce
replace BAD_ARGUMENTS error with INVALID_IDENTIFIER 2024-01-12 11:34:35 -05:00
Dmitry Novik
a3dcd350dd
Merge pull request #58723 from ClickHouse/fix-analyzer-count-distinct-pass
Analyzer: Fix LOGICAL_ERROR in CountDistinctPass
2024-01-12 09:06:39 +01:00
Dmitry Novik
f41c6b0e6b Analyzer: Fix LOGICAL_ERROR in CountDistinctPass 2024-01-11 19:11:42 +00:00
yariks5s
567e5d378e fixes 2024-01-11 15:43:17 +00:00
yariks5s
92c6eba922 avoiding only for paste join now 2024-01-10 18:18:50 +00:00
Robert Schulze
52d75ab68f
Merge pull request #58458 from rschu1ze/replace-std_regexp-by-re2
Replace `std::regex` by re2
2024-01-10 17:46:54 +01:00
yariks5s
f656a6d799 errors fix 2024-01-10 15:16:53 +00:00
Yarik Briukhovetskyi
d1f00b5b22
Merge branch 'master' into paste_join_aliases_requirement 2024-01-10 15:22:03 +01:00
yariks5s
7a1c1a3b62 analyzer implementation 2024-01-10 13:38:46 +00:00
vdimir
2c68bdecd8
add comment to check_nested_column_not_in_using 2024-01-10 13:35:33 +00:00
Dmitry Novik
b2bf1372aa
Merge pull request #58457 from ClickHouse/analyzer-fuzz-low-condinality
Analyzer: Fix LOGICAL_ERROR with LowCardinality
2024-01-10 11:06:42 +01:00
vdimir
25268fb863
Analyzer: Fix resolving subcolumns in JOIN 2024-01-09 19:05:44 +00:00
Dmitry Novik
629d4b921e Fix style 2024-01-09 15:38:04 +00:00
Dmitry Novik
11280cbde1 Merge remote-tracking branch 'origin/master' into storage-merge-aliases-analyzer 2024-01-09 15:10:59 +00:00
Jiebin Sun
22aadc7f5d add the default action for get() 2024-01-09 00:39:39 +08:00
Kruglov Pavel
b947609b8e
Merge branch 'master' into variant-data-type 2024-01-08 15:04:51 +01:00
flynn
e2588850f6 Fix conflict 2024-01-08 11:36:19 +00:00
Jiebin Sun
7bbeaf461f
Merge pull request #6 from jiebinn/master
Update to the latest upstream
2024-01-08 15:49:50 +08:00
Jiebin Sun
dfc8e79511 Add the Analyzer implementation for the RewriteSumFunctionWithSumAndCount
Signed-off-by: Jiebin Sun <jiebin.sun@intel.com>
2024-01-08 23:58:13 +08:00
Robert Schulze
f553b55e3a
Merge remote-tracking branch 'rschu1ze/master' into regex-std-re2 2024-01-07 22:31:35 +00:00
Robert Schulze
8e804487f3
Some fixups 2024-01-07 22:28:08 +00:00
Smita Kulkarni
16e682e0ae Merge branch 'master' into Support_parameterized_view_with_analyzer 2024-01-07 16:38:01 +01:00
Smita Kulkarni
4be844509e Merge branch 'master' into Support_parameterized_view_with_analyzer 2024-01-05 13:58:07 +01:00
Nikolai Kochetov
80c1e59d3f Merge branch 'master' into analyzer-execution-names 2024-01-05 09:31:25 +00:00