fix(sql): allow quotes around dedup keys (#3691)

core/src/main/java/io/questdb/griffin/SqlCompilerImpl.java

@@ -914,9 +914,9 @@ public class SqlCompilerImpl implements SqlCompiler, Closeable {
             int columnListPos = lexer.lastTokenPosition();
 
             while (tok != null && !Chars.equals(tok, ')')) {
-                final CharSequence columnName = tok;
-
                 validateLiteral(lexer.lastTokenPosition(), tok);
+                final CharSequence columnName = GenericLexer.unquote(tok);
+
                 int colIndex = tableMetadata.getColumnIndexQuiet(columnName);
                 if (colIndex < 0) {
                     throw SqlException.position(lexer.lastTokenPosition()).put("deduplicate key column not found [column=").put(columnName).put(']');

core/src/main/java/io/questdb/griffin/SqlParser.java

@@ -639,8 +639,8 @@ public class SqlParser {
                 int columnListPos = lexer.lastTokenPosition();
 
                 while (tok != null && !Chars.equals(tok, ')')) {
-                    final CharSequence columnName = tok;
                     validateLiteral(lexer.lastTokenPosition(), tok);
+                    final CharSequence columnName = GenericLexer.unquote(tok);
 
                     int colIndex = model.getColumnIndex(columnName);
                     if (colIndex < 0) {

core/src/test/java/io/questdb/test/griffin/CreateTableDedupTest.java

@@ -162,6 +162,37 @@ public class CreateTableDedupTest extends AbstractCairoTest {
         });
     }
 
+    @Test
+    public void testCreateTableWithDoubleQuotes() throws Exception {
+        String tableName = testName.getMethodName() + " a 欢迎回来 to you";
+        assertMemoryLeak(ff, () -> {
+            ddl(
+                    "CREATE TABLE '" + tableName + "' (\n" +
+                            "  Status SYMBOL capacity 16 CACHE,\n" +
+                            "  \"Reported time\" TIMESTAMP\n" +
+                            "  ) timestamp (\"Reported time\") PARTITION BY DAY WAL  DEDUP UPSERT KEYS(\"Reported time\");"
+            );
+            try (TableWriter writer = getWriter(tableName)) {
+                Assert.assertTrue(writer.getMetadata().isDedupKey(1));
+            }
+            assertSql(
+                    "column\ttype\tindexed\tindexBlockCapacity\tsymbolCached\tsymbolCapacity\tdesignated\tupsertKey\n" +
+                            "Status\tSYMBOL\tfalse\t256\ttrue\t16\tfalse\tfalse\n" +
+                            "Reported time\tTIMESTAMP\tfalse\t0\tfalse\t0\ttrue\ttrue\n",
+                    "SHOW COLUMNS FROM '" + tableName + '\''
+            );
+            ddl("alter table '" + tableName + "' DEDUP DISABLE;");
+            ddl("alter table '" + tableName + "' DEDUP ENABLE UPSERT KEYS(\"Reported time\");");
+            drainWalQueue();
+            assertSql(
+                    "column\ttype\tindexed\tindexBlockCapacity\tsymbolCached\tsymbolCapacity\tdesignated\tupsertKey\n" +
+                            "Status\tSYMBOL\tfalse\t256\ttrue\t16\tfalse\tfalse\n" +
+                            "Reported time\tTIMESTAMP\tfalse\t0\tfalse\t0\ttrue\ttrue\n",
+                    "SHOW COLUMNS FROM '" + tableName + '\''
+            );
+        });
+    }
+
     @Test
     public void testDedupEnabledTimestampOnly() throws Exception {
         String tableName = testName.getMethodName();