thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-24 00:22:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix "parseDateTimeBestEffort" for strings in RFC-2822 when day of week is Thuesday or Thursday #10082

Browse Source
This commit is contained in:
Alexey Milovidov 2020-04-12 22:34:04 +03:00
parent 8cb4dd275a
commit 142087c4f7
3 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/IO/parseDateTimeBestEffort.cpp
View File

@ -371,7 +371,10 @@ ReturnType parseDateTimeBestEffortImpl(
{ {
char c = *in.position(); char c = *in.position();
if (c == ' ' || c == 'T') /// 'T' is a separator between date and time according to ISO 8601.
/// But don't skip it if we didn't read the date part yet, because 'T' is also a prefix or 'Tue' and 'Thu'.
if (c == ' ' || (c == 'T' && year && !has_time))
{ {
++in.position(); ++in.position();
} }

2
tests/queries/0_stateless/01123_parse_date_time_best_effort_even_more.reference Normal file
View File

@ -0,0 +1,2 @@
2018-08-18 07:22:16
2018-08-16 07:22:16

2
tests/queries/0_stateless/01123_parse_date_time_best_effort_even_more.sql Normal file
View File

@ -0,0 +1,2 @@
SELECT toTimeZone(parseDateTimeBestEffort('Thu, 18 Aug 2018 07:22:16 GMT'), 'UTC');
SELECT toTimeZone(parseDateTimeBestEffort('Tue, 16 Aug 2018 07:22:16 GMT'), 'UTC');