[MCOL-1197] CPIMPORT fails and Abandoned when data row is too BIG to Handle Created: 2018-02-05 Updated: 2018-05-22 Resolved: 2018-05-22 |
|
| Status: | Closed |
| Project: | MariaDB ColumnStore |
| Component/s: | cpimport |
| Affects Version/s: | 1.1.2 |
| Fix Version/s: | 1.1.5 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Aziz Vahora | Assignee: | Daniel Lee (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | centos6 | ||
| Attachments: |
|
| Sprint: | 2018-08, 2018-09, 2018-10, 2018-11 |
| Description |
|
terminate called after throwing an instance of 'std::runtime_error' mcsadmin restartSystem when cpimport fails, if does not properly rollback the transaction leaving the table locked and no way to kill the PID except to restart the system. |
| Comments |
| Comment by Andrew Hutchings (Inactive) [ 2018-02-05 ] | |||
|
Hi, Do you have a test case for this? Usually you would hit a buffer error with a rollback before hitting this. | |||
| Comment by Aziz Vahora [ 2018-02-07 ] | |||
|
Hey Andrew, create table test (id int,listofitems text) engine=innodb; insert a row into test with very very large text, usually more than 1 million characters. select from test table | cpimport -m1 -e 10 rtt test -s '\t' -f STDIN This is where cpimport fails and process never releases lock on the test_cs table. | |||
| Comment by Andrew Hutchings (Inactive) [ 2018-02-07 ] | |||
|
Confirmed. Test case: 1. gunzip the file MCOL-1197.sql.gz
3. /usr/local/mariadb/columnstore/mysql/bin/mysql -uroot --host=127.0.0.1 test < ~/tmp/
5. Execute the following:
It will fail as described. Increasing cpimport's buffers does not seem to help. | |||
| Comment by Andrew Hutchings (Inactive) [ 2018-05-09 ] | |||
|
Made cpimport's '-c' flag work for this. If < 1MB it sets it to 1MB, otherwise it will use the selected size. For QA. See the third comment. To make it work after the patch add '-c 8388608' to cpimport. | |||
| Comment by Daniel Lee (Inactive) [ 2018-05-22 ] | |||
|
Build verified: 1.1.5-1 source /root/columnstore/mariadb-columnstore-server Merge pull request #112 from mariadb-corporation/davidhilldallas-patch-3 update to 1.1.5 /root/columnstore/mariadb-columnstore-server/mariadb-columnstore-engine Merge pull request #475 from drrtuy/ Reproduced the issue in 1.1.4-1 and verified the fix using the -c option. |