thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-09-20 08:40:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #38074 from ClickHouse/bug-with-fill-extra

Browse Source 
Inconsistency in ORDER BY ... WITH FILL feature.
This commit is contained in:
Yakov Olkhovskiy 2022-06-15 08:19:55 -04:00 committed by GitHub
commit e604d31feb
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 1 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Interpreters/FillingRow.cpp
View File

@ -76,7 +76,7 @@ bool FillingRow::next(const FillingRow & to_row)
auto next_value = row[pos];
getFillDescription(pos).step_func(next_value);
if (less(to_row.row[pos], next_value, getDirection(pos)))
if (less(to_row.row[pos], next_value, getDirection(pos)) || equals(next_value, getFillDescription(pos).fill_to))
return false;
row[pos] = next_value;

5
tests/queries/0_stateless/00995_order_by_with_fill.reference
View File

@ -376,11 +376,6 @@
2019-05-03 4
2019-05-03 1
2019-05-03 -2
2019-05-01 10
2019-05-01 7
2019-05-01 4
2019-05-01 1
2019-05-01 -2
*** date WITH FILL TO 2019-06-23 STEP 3, val WITH FILL FROM -10 STEP 2
2019-05-07 -10
2019-05-07 -8

9
tests/queries/0_stateless/02019_multiple_weird_with_fill.reference
View File

@ -34,12 +34,3 @@
6 -4
6 -3
6 -2
7 -10
7 -9
7 -8
7 -7
7 -6
7 -5
7 -4
7 -3
7 -2

4
tests/queries/0_stateless/02112_with_fill_interval.reference
View File

@ -103,10 +103,6 @@
2020-04-01 2 0
2020-04-01 3 0
2020-04-01 4 0
2020-05-01 1 0
2020-05-01 2 0
2020-05-01 3 0
2020-05-01 4 0
1970-01-04
1970-01-03
1970-01-02