Better Error Messages

[nbdamian]

Could I suggest error messages be similar to sql, for example:
If a column you are querying does not exist, you would error log:
Unknown column 'columnname' in 'tablename' or Column 'columnname' doesn't exist in 'tablename'

Also, a lot of errors are not being handled and return javascript errors, perhaps we should go through some common errors we know of and then add better error logging to them.

[mathiasrw]

Really good idea. Do you know if there is a standardised SQL error list?

perhaps we should go through some common errors we know of and then add better error logging to them

Do you have any inputs which ones to start with?

[nbdamian]

I'm not sure there are any global standards for sql errors, but I would expect oracle to be a good choice.
https://docs.oracle.com/cd/B28359_01/server.111/b28278/toc.htm

[nbdamian]

I would say definitely, start with the basics, ones that happen with incorrect syntax, or missing something important.
E.G
Incorrect Column Name / Column does not exists
Missing table in FROM
Missing parenthesis

[mathiasrw]

The parenthesis is tricky as its part of the parser generated from JISON

Now - I got the most common errors with error in table name and error in column name

> alasql('create table abc (a int)')
1

> alasql('select a from abc')
[]

> alasql('select b from abc')
Uncaught Error: Column does not exists: b

> alasql('select a from abcd')
Uncaught Error: Table does not exists: abcd

Please try it out in the last version of https://github.com/agershun/alasql/blob/develop/dist/alasql.js

[nbdamian]

Perfect, although I am unsure whether it was caused by this update, but it seems that it is now required to wrap any columns with ``.
If you don't wrap them it returns:

Uncaught Error: Column does not exists: 

Is this intentional?

Version: v0.2.5-develop-1262

[mathiasrw]

Hmm, what sql are you using. I cant replicate it:

// alasql.version: 0.2.5-develop-1262

> alasql('create table abc (a int)')
1

> alasql('insert into abc values(2), (5)')
2

> alasql('column of select a from abc')
[2, 5]

> alasql('column of select b from abc')
Uncaught Error: Column does not exists: b

[nbdamian]

Strange... I wonder what was causing it. Ignore then ;)