[MCOL-533] table_usage() has a calculation error Created: 2017-01-25 Updated: 2017-03-22 Resolved: 2017-03-22 |
|
| Status: | Closed |
| Project: | MariaDB ColumnStore |
| Component/s: | ExeMgr |
| Affects Version/s: | 1.0.7 |
| Fix Version/s: | 1.0.8, 1.1.0 |
| Type: | Bug | Priority: | Major |
| Reporter: | Andrew Hutchings (Inactive) | Assignee: | Daniel Lee (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Sprint: | 2017-2, 2017-3, 2017-4, 2017-5, 2017-6 |
| Description |
|
The table_usage() procedure adds entries multiple times under some scenarios which can cause significant errors. The procedure needs re-writing. |
| Comments |
| Comment by Andrew Hutchings (Inactive) [ 2017-01-25 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Recommend backporting this to 1.0 if we do another release. Need to add a warning in the documentation that this shouldn't be run by more than one client at the same time (the new version of table_usage will have problems if you try). Anyone who is hit with this problem can fix it on their own installation by manually piping this version of columnstore_info.sql into their mariadb client as root. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Andrew Hutchings (Inactive) [ 2017-01-25 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Second pull request adds safety locks as discussed with David Hall. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Daniel Lee (Inactive) [ 2017-03-02 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Build tested: Merge pull request #31 from jbfavre/fix_deb_package_dependency [root@localhost mariadb-columnstore-server]# cd mariadb-columnstore-engine/ change the check for prompt back to the previous code Tested with concurrent sessions. While session 1 is running, session 2 return the following, which is the expected behavior. MariaDB [tpch10c]> call columnstore_info.table_usage(NULL,NULL); "acquiring" spelling is incorrect.
-------------
------------- Query OK, 0 rows affected (5.37 sec) Results are in bytes only. It should display in mb, gb. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Daniel Lee (Inactive) [ 2017-03-02 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Per my last comment | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Andrew Hutchings (Inactive) [ 2017-03-02 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Fix for units and spelling error that Daniel spotted | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Daniel Lee (Inactive) [ 2017-03-03 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Build verified: Github source [root@localhost mariadb-columnstore-server]# git show Merge pull request #31 from jbfavre/fix_deb_package_dependency [root@localhost mariadb-columnstore-engine]# git show Merge pull request #129 from mariadb-corporation/ Verified the latest fixes. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by David Thompson (Inactive) [ 2017-03-07 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
re-opening to backport to 1.0.8 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Andrew Hutchings (Inactive) [ 2017-03-07 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
New pull request for backport to 1.0 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Daniel Lee (Inactive) [ 2017-03-21 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Tested in 1.0.8-1 on a 1um2pm stack MariaDB [(none)]> use tpch10c There are crit.log and err.log files in all three nodes. Therefore, no errors have been logged. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Andrew Hutchings (Inactive) [ 2017-03-22 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
This problem is the one in | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Daniel Lee (Inactive) [ 2017-03-22 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Build verified: 1.0.8-1 mcsadmin> getsoftware Name : mariadb-columnstore-platform MariaDB [(none)]> call columnstore_info.table_usage(NULL,NULL);
-------------
------------- Query OK, 0 rows affected (0.92 sec) MariaDB [(none)]> call columnstore_info.table_usage(NULL,NULL); On a 1um2pm stack, it is hard to see the locking mechanism working, since the query does not take long to execute. Testing with two session right after the ColumnStore has been restarted has a higher chance of see the lock. Build verified: 1.1.0 GitHub source MariaDB [(none)]> call columnstore_info.table_usage(NULL,NULL);
-------------
------------- MariaDB [(none)]> call columnstore_info.table_usage(NULL,NULL); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Daniel Lee (Inactive) [ 2017-03-22 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Closed per my last comment. |