[MCOL-3809] CLONE - TIMEDIFF() returns incorrect result Created: 2020-02-19  Updated: 2020-03-23  Resolved: 2020-03-23

Status: Closed
Project: MariaDB ColumnStore
Component/s: None
Affects Version/s: 1.4.0
Fix Version/s: 1.2.6

Type: Bug Priority: Major
Reporter: Patrick LeBlanc (Inactive) Assignee: Daniel Lee (Inactive)
Resolution: Fixed Votes: 0
Labels: None


 Description   

Build tested: 1.4.0-1

[dlee@master centos7]$ cat gitversionInfo.txt
server commit:
67452bc
engine commit:
4d2a159

Autopilot test case: features.functions

ColumnStore returned:

18 select cidx, CTIME, TIMEDIFF(CTIME,'2007-02-28 22:23:0') from datatypetestm order by cidx
19 --------------
20
21 cidx CTIME TIMEDIFF(CTIME,'2007-02-28 22:23:0')
22 1 13:00:00 -838:59:59
23 1 row in set

40 select cidx, CTIME from datatypetestm where TIMEDIFF(CTIME,'2007-02-28 22:23:0') <> 0 order by cidx
41 --------------
42
43 cidx CTIME
44 1 13:00:00
45 1 row in set

reference returned:

18 select cidx, CTIME, TIMEDIFF(CTIME,'2007-02-28 22:23:0') from datatypetestm order by cidx
19 --------------
20
21 cidx CTIME TIMEDIFF(CTIME,'2007-02-28 22:23:0')
22 1 13:00:00 NULL
23 1 row in set

40 select cidx, CTIME from datatypetestm where TIMEDIFF(CTIME,'2007-02-28 22:23:0') <> 0 order by cidx
41 --------------
42
43 Empty set

This test works in 1.2.4-1

 Comments   
Comment by Daniel Lee (Inactive) [ 2020-03-16 ]

Build verified: 1.2.6-1 BB

engine commit:
d4173ef

This build still returned incorrect result

Comment by Patrick LeBlanc (Inactive) [ 2020-03-17 ]

I must have created this ticket by mistake. In my notes from the merge, it says I had to revert the commits for this and for mcol-3597 because it broke bug3333.sql in our regression suite.

I re-assigned it to Jose to look into, who was the original author of MCOL-3474, which is what this ticket was cloned from.

Comment by Patrick LeBlanc (Inactive) [ 2020-03-18 ]

It should match the functionality in 1.4 and 1.5 now. I'll create another ticket to follow up on the remaining bug3333 problem.

Comment by Daniel Lee (Inactive) [ 2020-03-23 ]

Build verified: 1.2.6-1 source

/root/ColumnStore/buildColumnstoreFromGithubSource/mariadb-columnstore-server
commit d3294cf9c18663673c62532dddaa11369ba184d0
Merge: 2ef719d c5be945
Author: Andrew Hutchings <andrew@linuxjedi.co.uk>
Date: Thu Oct 31 15:06:48 2019 +0000

Merge pull request #166 from drrtuy/MCOL-894_5

MCOL-894 Add a variable to control a number of parallel sorting threads.

/root/ColumnStore/buildColumnstoreFromGithubSource/mariadb-columnstore-server/mariadb-columnstore-engine
commit c3dc4ebb847e5399d4edc28754b3200e83253774
Merge: f4880b6 22a25af
Author: Patrick LeBlanc <43503225+pleblanc1976@users.noreply.github.com>
Date: Fri Mar 20 16:12:25 2020 -0500

Merge pull request #1120 from jmrojas2332/develop-1.2

MCOL-3809 Missed change when porting from 1.4 to 1.2

Generated at Thu Feb 08 02:45:36 UTC 2024 using Jira 8.20.16#820016-sha1:9d11dbea5f4be3d4cc21f03a88dd11d8c8687422.