MySQL Error during 'v.in.ogr' in GRASS 7 SVN (2014-01-31)

[justinzane]

It seems that there is a failure to properly quote the CREATE TABLE command. Simply adding backticks "`" around the table name and all column names allows the command to execute.

This looks like create table `natural` (`cat` integer, `osm_id` varchar (11), `name` varchar (48), `type` varchar (16));.

Additionally, even though the CREATE TABLE failed and therefore the import failed, a map is created and cannot be removed since its backing data table never existed.

So, two separate issues need to be fixed. First the quoting. Second is the need to wait until the import has succeeded before allocating a 'Map'.

v.in.ogr dsn=/home/justin/downloads/osm_CA/natural.shp
...
DBMI-MySQL driver error:
Unable to execute:
create table natural (cat integer, osm_id varchar ( 11 ),
name varchar ( 48 ), type varchar ( 16 ))
You have an error in your SQL syntax; check the manual that
corresponds to your MySQL server version for the right
syntax to use near 'natural (cat integer, osm_id varchar (
11 ), name varchar ( 48 ), type varchar (' at line 1
ERROR: Unable to create table: 'create table natural (cat integer, osm_id varchar ( 11 ), name varchar ( 48 ), type varchar ( 16 ))'

[justinzane]

Note that this may properly belong under "Database".

Also, in the off chance it is wanted, I can provide all configure/build logs.

[neteler]

According to

• ​http://dev.mysql.com/doc/refman/5.7/en/reserved-words.html
• ​http://dev.mysql.com/doc/refman/5.7/en/identifiers.html
• ​http://stackoverflow.com/questions/12658763/why-did-i-have-to-use-backticks-around-the-table-name-in-my-mysql-query

backticks are required if the identifier is a reserved word, contains white space, etc.

The word "NATURAL" is a reserved word in MySQL (see Table 9.2 in the first link). To solve this conflict for you, you can use the "cnames" parameter of v.in.ogr to change the column names during import. The GRASS DB drivers should not try to automatically launder names.

To give more help in the manual, I have added some remarks in r59058 (backported in r59059 and r59060).

Closing.