{"action":"created","issue":{"url":"https://api.github.com/repos/CartoDB/cartodb/issues/3884","labels_url":"https://api.github.com/repos/CartoDB/cartodb/issues/3884/labels{/name}","comments_url":"https://api.github.com/repos/CartoDB/cartodb/issues/3884/comments","events_url":"https://api.github.com/repos/CartoDB/cartodb/issues/3884/events","html_url":"https://github.com/CartoDB/cartodb/pull/3884","id":83587012,"number":3884,"title":"Adds more reserved keywords ","user":{"login":"iriberri","id":1730320,"avatar_url":"https://avatars.githubusercontent.com/u/1730320?v=3","gravatar_id":"","url":"https://api.github.com/users/iriberri","html_url":"https://github.com/iriberri","followers_url":"https://api.github.com/users/iriberri/followers","following_url":"https://api.github.com/users/iriberri/following{/other_user}","gists_url":"https://api.github.com/users/iriberri/gists{/gist_id}","starred_url":"https://api.github.com/users/iriberri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/iriberri/subscriptions","organizations_url":"https://api.github.com/users/iriberri/orgs","repos_url":"https://api.github.com/users/iriberri/repos","events_url":"https://api.github.com/users/iriberri/events{/privacy}","received_events_url":"https://api.github.com/users/iriberri/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-06-01T16:22:22Z","updated_at":"2015-06-24T08:25:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/CartoDB/cartodb/pulls/3884","html_url":"https://github.com/CartoDB/cartodb/pull/3884","diff_url":"https://github.com/CartoDB/cartodb/pull/3884.diff","patch_url":"https://github.com/CartoDB/cartodb/pull/3884.patch"},"body":"Fixes #2718 \r\n\r\nI've added a selection of keywords from the complete list of reserved and non-reserved Postgres keywords (below). Not all of them fail in the import process, so I have selected the most common ones which fail.\r\n\r\n`all and any array as asc before both cast desc end fetch for from group intersect not offset on or select table then to union when where with`\r\n\r\nComplete list:\r\n\r\n````\r\nPASSWORD, MODE, ABORT, ABSOLUTE, ACCESS, ACTION, ADD,\r\n ADMIN, AFTER, AGGREGATE, ALL, ALSO, ALTER, ANALYSE,\r\n ANALYZE, AND, ANY, ARRAY, AS, ASC, ASSERTION, ASSIGNMENT,\r\n ASYMMETRIC, AT, BACKWARD, BEFORE, BEGIN, BOTH, BY, CACHE,\r\n CALLED, CASCADE, CASE, CAST, CHAIN, CHARACTERISTICS, CHECK,\r\n CHECKPOINT, CLASS, CLOSE, CLUSTER, COLLATE, COLUMN,\r\n COMMENT, COMMIT, COMMITTED, CONNECTION, CONSTRAINT,\r\n CONSTRAINTS, CONVERSION, COPY, CREATE, CREATEDB, \r\nCREATEROLE, CREATEUSER, CSV, CURRENT_DATE,\r\n CURRENT_ROLE, CURRENT_TIME, CURRENT_TIMESTAMP,\r\n CURRENT_USER, CURSOR, CYCLE, DATABASE, DAY, DEALLOCATE,\r\n DECLARE, DEFAULT, DEFAULTS, DEFERRABLE, DEFERRED,\r\n DEFINER, DELETE, DELIMITER, DELIMITERS, DESC, DISABLE,\r\n DISTINCT, DO, DOMAIN, DOUBLE, DROP, EACH, ELSE, ENABLE,\r\n ENCODING, ENCRYPTED, END, ESCAPE, EXCEPT, EXCLUDING,\r\n EXCLUSIVE, EXECUTE, EXPLAIN, EXTERNAL, FALSE, FETCH,\r\n FIRST, FOR, FORCE, FOREIGN, FORWARD, FROM, FUNCTION,\r\n GLOBAL, GRANT, GRANTED, GROUP, HANDLER, HAVING,\r\n HEADER, HOLD, HOUR, IMMEDIATE, IMMUTABLE, IMPLICIT,\r\n IN, INCLUDING, INCREMENT, INDEX, INHERIT, INHERITS, INITIALLY,\r\n INPUT, INSENSITIVE, INSERT, INSTEAD, INTERSECT, INTO, INVOKER,\r\n ISOLATION, KEY, LANCOMPILER, LANGUAGE, LARGE, LAST,\r\n LEADING, LEVEL, LIMIT, LISTEN, LOAD, LOCAL, LOCALTIME,\r\n LOCALTIMESTAMP, LOCATION, LOCK, LOGIN, MATCH, MAXVALUE,\r\n MINUTE, MINVALUE, MONTH, MOVE, NAMES, NEW, NEXT, NO,\r\n NOCREATEDB, NOCREATEROLE, NOCREATEUSER, NOINHERIT,\r\n NOLOGIN, NOSUPERUSER, NOT, NOTHING, NOTIFY, NOWAIT, NULL,\r\n OBJECT, OF, OFF, OFFSET, OIDS, OLD, ON, ONLY, OPERATOR,\r\n OPTION, OR, ORDER, OWNER, PARTIAL, PLACING, PREPARE,\r\n PREPARED, PRESERVE, PRIMARY, PRIOR, PRIVILEGES, PROCEDURAL,\r\n PROCEDURE, QUOTE, READ, RECHECK, REFERENCES, REINDEX,\r\n RELATIVE, RELEASE, RENAME, REPEATABLE, REPLACE, RESET,\r\n RESTART, RESTRICT, RETURNS, REVOKE, ROLE, ROLLBACK,\r\n ROWS, RULE, SAVEPOINT, SCHEMA, SCROLL, SECOND, SECURITY,\r\n UNTIL, SELECT, SEQUENCE, SERIALIZABLE, SESSION, SESSION_USER,\r\n SET, SHARE, SHOW, SIMPLE, SOME, STABLE, START, STATEMENT,\r\n STATISTICS, STDIN, STDOUT, STORAGE, STRICT, SUPERUSER,\r\n SYMMETRIC, SYSID, SYSTEM, TABLE, TABLESPACE, TEMP, TEMPLATE,\r\n TEMPORARY, THEN, TO, TOAST, TRAILING, TRANSACTION, TRIGGER,\r\n TRUE, TRUNCATE, TRUSTED, TYPE, UNCOMMITTED, UNENCRYPTED,\r\n UNION, UNIQUE, UNKNOWN, UNLISTEN, UPDATE, USER, USING,\r\n VACUUM, VALID, VALIDATOR, VALUES, VARYING, VIEW, VOLATILE,\r\n WHEN, WHERE, WITH, WITHOUT, WORK, WRITE, YEAR, ZONE\r\n````\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/CartoDB/cartodb/issues/comments/114782446","html_url":"https://github.com/CartoDB/cartodb/pull/3884#issuecomment-114782446","issue_url":"https://api.github.com/repos/CartoDB/cartodb/issues/3884","id":114782446,"user":{"login":"iriberri","id":1730320,"avatar_url":"https://avatars.githubusercontent.com/u/1730320?v=3","gravatar_id":"","url":"https://api.github.com/users/iriberri","html_url":"https://github.com/iriberri","followers_url":"https://api.github.com/users/iriberri/followers","following_url":"https://api.github.com/users/iriberri/following{/other_user}","gists_url":"https://api.github.com/users/iriberri/gists{/gist_id}","starred_url":"https://api.github.com/users/iriberri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/iriberri/subscriptions","organizations_url":"https://api.github.com/users/iriberri/orgs","repos_url":"https://api.github.com/users/iriberri/repos","events_url":"https://api.github.com/users/iriberri/events{/privacy}","received_events_url":"https://api.github.com/users/iriberri/received_events","type":"User","site_admin":false},"created_at":"2015-06-24T08:25:43Z","updated_at":"2015-06-24T08:25:43Z","body":"\"right\" should be here too. I'll block this PR until @Kartones works in the importer sprint so that we could check the test that is failing, which makes no sense for me but we'd need to check :)"}}