[MCOL-1219] Column name can not start with numeric value Created: 2018-02-19  Updated: 2018-05-11  Resolved: 2018-05-11

Status: Closed
Project: MariaDB ColumnStore
Component/s: None
Affects Version/s: 1.1.2
Fix Version/s: 1.2.0

Type: Bug Priority: Minor
Reporter: VAROQUI Stephane Assignee: Daniel Lee (Inactive)
Resolution: Fixed Votes: 0
Labels: None

Epic Link: ColumnStore Compatibility Improvements
Sprint: 2018-05, 2018-06, 2018-07, 2018-08, 2018-09, 2018-10

 Description   

CREATE TABLE t ( `4RM` char(1)) engine=columnstore;
ERROR 1178 (42000) at line 1: The storage engine for the table doesn't support The syntax or the data type(s) is not supported by Columnstore. Please check the Columnstore syntax guide for supported syntax or data types.

 Comments   
Comment by Andrew Hutchings (Inactive) [ 2018-02-19 ]

Assigned to Roman as he has been working on similar bugs for 1.2

Comment by Roman [ 2018-02-20 ]

I didn't allow numerics as a first symbol for db objects in the first place, since this KB article, mentioned below, says it is impractical and could overlap with hex constants. Will fix it shortly.

https://mariadb.com/kb/en/library/identifier-names/

Comment by VAROQUI Stephane [ 2018-02-20 ]

In a more generic way i think it would really benefit column store to support index and mediumint under the hood by just doing nothing for the index and stroring mediumint as int , this would just allow most alter table engine=columnstore without any rewrite and backward engine=innodb should be doable without lost of DDL component

Comment by Andrew Hutchings (Inactive) [ 2018-02-20 ]

MEDIUMINT is a separate issue tracked in MCOL-270 (feel free to vote on it to increase priority), we have a plan for this. Please keep this ticket for tracking the numeric start DDL syntax issue.

Comment by VAROQUI Stephane [ 2018-02-20 ]

Thanks for pointing voted, keep on good work heroes

Comment by Roman [ 2018-02-28 ]

Here are the testing results, I got with the tiny fix.

Comment by Roman [ 2018-02-28 ]

Please take a look at the PR.

Comment by Daniel Lee (Inactive) [ 2018-05-11 ]

Build verified: 1.2.0 source

/root/columnstore/mariadb-columnstore-server
commit 4334641df0df040e0f53332c11dc8e29dc34b8b7
Merge: 960853c cd5e845
Author: David.Hall <david.hall@mariadb.com>
Date: Mon Apr 9 13:23:44 2018 -0500

Merge pull request #107 from mariadb-corporation/dev-merge-up-20180409

Dev merge up 20180409

/root/columnstore/mariadb-columnstore-server/mariadb-columnstore-engine
commit d8d386b960cbfa77d3077fc84f31e076f867efaa
Merge: 24b8c79 305bae1
Author: David.Hall <david.hall@mariadb.com>
Date: Wed May 9 15:13:47 2018 -0500

Merge pull request #468 from mariadb-corporation/MCOL-1402

MCOL-1402 Fix addtime/subtime

Verified with query,, subquery, join, DML, etc.

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