fix: accept VISIBLE/INVISIBLE index and column attributes in CREATE TABLE and CREATE INDEX (#2449)

MySQL 8.0 invisible indexes (KEY idx (col) INVISIBLE) and invisible columns
failed to parse inside CREATE TABLE, and CREATE INDEX ... INVISIBLE failed
too, because CreateParameter() did not accept the K_VISIBLE/K_INVISIBLE
tokens. The ALTER TABLE productions already support them (#2076 was fixed
for ALTER TABLE only, via #2234); this covers the CREATE statements, which
share CreateParameter(). Tests cover index and column visibility, mixing
visibility with other index options in either order, and quoted columns
named visible/invisible.

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

Author: aparajon

src/main/jjtree/net/sf/jsqlparser/parser/JSqlParserCC.jjt

@@ -11056,6 +11056,7 @@ List<String> CreateParameter():
             | tk=<K_DESC> | tk=<K_TRUE> | tk=<K_FALSE> | tk=<K_PARALLEL> | tk=<K_BINARY> | tk=<K_START> | tk=<K_ORDER>
             | tk=<K_TIME_KEY_EXPR> | tk=<K_RAW> | tk=<K_HASH> | tk=<K_FIRST> | tk=<K_LAST> | tk = <K_SIGNED>  | tk = <K_UNSIGNED>
             | tk=<K_ENGINE> | tk=<K_IDENTITY> | tk=<K_MATERIALIZED> | tk=<K_SAMPLE> | tk=<K_ALWAYS>
+            | tk=<K_VISIBLE> | tk=<K_INVISIBLE>
             | tk="="
         )
         { param.add(tk.image); }

src/test/java/net/sf/jsqlparser/statement/create/CreateIndexTest.java

@@ -168,4 +168,10 @@ public void testCreateIndexWithFunctionalKeyParts() throws JSQLParserException {
 
         assertSqlCanBeParsedAndDeparsed(statement);
     }
+
+    @Test
+    public void testCreateIndexVisibility() throws JSQLParserException {
+        assertSqlCanBeParsedAndDeparsed("CREATE INDEX idx_a ON t1 (a) INVISIBLE", true);
+        assertSqlCanBeParsedAndDeparsed("CREATE INDEX idx_a ON t1 (a) VISIBLE", true);
+    }
 }

src/test/java/net/sf/jsqlparser/statement/create/CreateTableTest.java

@@ -1162,4 +1162,48 @@ void testWithCatalog() throws JSQLParserException {
         assertEquals("session1", t.getSchemaName());
         assertEquals("a", t.getUnquotedName());
     }
+
+    @Test
+    void testCreateTableIndexVisibility() throws JSQLParserException {
+        String sqlStr = "CREATE TABLE `orders` (" +
+                "`id` bigint NOT NULL AUTO_INCREMENT" +
+                ", `status` varchar (32) NOT NULL" +
+                ", `quote_id` varchar (191) NOT NULL" +
+                ", PRIMARY KEY (`id`)" +
+                ", KEY `idx_status` (`status`) INVISIBLE" +
+                ", UNIQUE KEY `idx_quote_id` (`quote_id`) VISIBLE" +
+                ") ENGINE = InnoDB";
+        CreateTable createTable = (CreateTable) assertSqlCanBeParsedAndDeparsed(sqlStr, true);
+
+        assertEquals(Arrays.asList("INVISIBLE"), createTable.getIndexes().get(1).getIndexSpec());
+        assertEquals(Arrays.asList("VISIBLE"), createTable.getIndexes().get(2).getIndexSpec());
+    }
+
+    @Test
+    void testCreateTableColumnVisibility() throws JSQLParserException {
+        String sqlStr =
+                "CREATE TABLE t1 (id bigint NOT NULL VISIBLE, secret varchar (10) INVISIBLE)";
+        CreateTable createTable = (CreateTable) assertSqlCanBeParsedAndDeparsed(sqlStr, true);
+
+        assertEquals(Arrays.asList("NOT", "NULL", "VISIBLE"),
+                createTable.getColumnDefinitions().get(0).getColumnSpecs());
+        assertEquals(Arrays.asList("INVISIBLE"),
+                createTable.getColumnDefinitions().get(1).getColumnSpecs());
+    }
+
+    @Test
+    void testCreateTableIndexVisibilityWithOtherIndexOptions() throws JSQLParserException {
+        assertSqlCanBeParsedAndDeparsed(
+                "CREATE TABLE t1 (a int, KEY idx_a (a) INVISIBLE COMMENT 'retiring')", true);
+        assertSqlCanBeParsedAndDeparsed(
+                "CREATE TABLE t1 (a int, KEY idx_a (a) COMMENT 'retiring' INVISIBLE)", true);
+        assertSqlCanBeParsedAndDeparsed(
+                "CREATE TABLE t1 (a int, KEY idx_a (a) USING BTREE INVISIBLE)", true);
+    }
+
+    @Test
+    void testCreateTableQuotedColumnNamedVisible() throws JSQLParserException {
+        assertSqlCanBeParsedAndDeparsed(
+                "CREATE TABLE t1 (`visible` int, `invisible` varchar (10))", true);
+    }
 }