[MCOL-344] Set test001 working_tpch1 to Tr mode Created: 2016-10-03 Updated: 2016-11-28 Resolved: 2016-11-28 |
|
| Status: | Closed |
| Project: | MariaDB ColumnStore |
| Component/s: | regression suite |
| Affects Version/s: | None |
| Fix Version/s: | 1.0.5 |
| Type: | Bug | Priority: | Major |
| Reporter: | Andrew Hutchings (Inactive) | Assignee: | David Hall (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||||||||||||||||||
| Sprint: | 2016-19, 2016-20, 2016-21, 2016-22, 2016-23 | ||||||||||||||||||||||||
| Description |
|
This is a continuation after Compare failed - working_tpch1/aggregation/bitop.sql Compare failed - working_tpch1/misc/bug2584.sql Compare failed - working_tpch1/qa_fe_cnxFunctions/addtime.sql Compare failed - working_tpch1/qa_fe_postProcessedFunctions/CONCAT_WS.SM.sql Compare failed - working_tpch1/view/view.sql Many of these could be due to a needed ref update. All need analysing ASAP. |
| Comments |
| Comment by Andrew Hutchings (Inactive) [ 2016-10-03 ] |
|
Analysis: working_tpch1/aggregation/* - update ref working_tpch1/misc/bug3881.sql - needs order by working_tpch1/qa_fe_cnxFunctions/bug3333.sql - bugs in time() handling working_tpch1/qa_fe_postProcessedFunctions/* - update refs working_tpch1/view/view.sql - I'll open required tickets tomorrow |
| Comment by Andrew Hutchings (Inactive) [ 2016-10-04 ] |
|
Only NULLIF.DM.sql and view.sql are things we should be concerned about for Beta. |
| Comment by Andrew Hutchings (Inactive) [ 2016-10-08 ] |
|
Pull request for the ref and query fixes can be found at: https://github.com/mariadb-corporation/mariadb-columnstore-regression-test/pull/2 |
| Comment by David Hall (Inactive) [ 2016-10-17 ] |
|
I've changed a bunch of the tests in working_tpch1_compareLogOnly so they compare properly. Please review and merge. We still need to change working_tpch1 to -r B, but I'm waiting on your opinion on what would break due to the changes you made. |
| Comment by Andrew Hutchings (Inactive) [ 2016-10-18 ] |
|
If we do set it back to 'B' I highly recommend 'B' in local mode setting 'Tr' instead of 'T' like it does now. I'll merge your fixes and leave this assigned to me pending outcome of the discussion. |
| Comment by Andrew Hutchings (Inactive) [ 2016-10-27 ] |
|
still some discussions and work to do |
| Comment by Andrew Hutchings (Inactive) [ 2016-11-24 ] |
|
Switched working_tpch1 back to B mode instead of Tr so that we don't use the ref files in local mode. Everything else should be good now I believe. |