Parse dictionary SETTINGS clause and output as Dictionary settings

Add proper parsing for SETTINGS clause in CREATE DICTIONARY statements.
Handle SETTINGS as a keyword token (not IDENT).

Changes:
- Add SETTINGS keyword handling in dictionary definition parsing
- Parse settings with or without parentheses
- Output as "Dictionary settings" (not "Set") in explain
- Update condition to include Settings in dictionary definition check
- Fixes 01268_dictionary_direct_layout stmt25, stmt26
- Also fixes: 01259_dictionary_custom_settings_ddl stmt6,
  01676_range_hashed_dictionary stmt5, 01681_cache_dictionary_simple_key stmt7,
  01760_polygon_dictionaries stmt17, 01765_hashed_dictionary_simple_key stmt7

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: kyleconroy

internal/explain/dictionary.go

@@ -92,7 +92,7 @@ func explainDictionaryDefinition(sb *strings.Builder, n *ast.DictionaryDefinitio
 
 	// SETTINGS
 	if len(n.Settings) > 0 {
-		fmt.Fprintf(sb, "%s Set\n", indent)
+		fmt.Fprintf(sb, "%s Dictionary settings\n", indent)
 	}
 }
 

parser/parser.go

@@ -3850,6 +3850,19 @@ func (p *Parser) parseCreateDictionary(create *ast.CreateQuery) {
 			}
 			continue
 		}
+		// Handle SETTINGS as a keyword token
+		if p.currentIs(token.SETTINGS) {
+			p.nextToken() // skip SETTINGS
+			// Parse dictionary settings: SETTINGS(key=value, ...) or SETTINGS key=value, ...
+			if p.currentIs(token.LPAREN) {
+				p.nextToken() // skip (
+				dictDef.Settings = p.parseSettingsList()
+				p.expect(token.RPAREN)
+			} else {
+				dictDef.Settings = p.parseSettingsList()
+			}
+			continue
+		}
 		if p.currentIs(token.IDENT) {
 			upper := strings.ToUpper(p.current.Value)
 			switch upper {
@@ -3873,9 +3886,13 @@ func (p *Parser) parseCreateDictionary(create *ast.CreateQuery) {
 				dictDef.Range = p.parseDictionaryRange()
 			case "SETTINGS":
 				p.nextToken() // skip SETTINGS
-				// Skip settings for now
-				for !p.currentIs(token.EOF) && !p.currentIs(token.SEMICOLON) && !p.isDictionaryClauseKeyword() {
-					p.nextToken()
+				// Parse dictionary settings: SETTINGS(key=value, ...) or SETTINGS key=value, ...
+				if p.currentIs(token.LPAREN) {
+					p.nextToken() // skip (
+					dictDef.Settings = p.parseSettingsList()
+					p.expect(token.RPAREN)
+				} else {
+					dictDef.Settings = p.parseSettingsList()
 				}
 			case "COMMENT":
 				p.nextToken() // skip COMMENT
@@ -3892,7 +3909,7 @@ func (p *Parser) parseCreateDictionary(create *ast.CreateQuery) {
 	}
 
 	// Only set dictionary definition if it has any content
-	if len(dictDef.PrimaryKey) > 0 || dictDef.Source != nil || dictDef.Lifetime != nil || dictDef.Layout != nil || dictDef.Range != nil {
+	if len(dictDef.PrimaryKey) > 0 || dictDef.Source != nil || dictDef.Lifetime != nil || dictDef.Layout != nil || dictDef.Range != nil || len(dictDef.Settings) > 0 {
 		create.DictionaryDef = dictDef
 	}
 }

parser/testdata/01259_dictionary_custom_settings_ddl/metadata.json

@@ -1,5 +1 @@
-{
-  "explain_todo": {
-    "stmt6": true
-  }
-}
+{}

parser/testdata/01268_dictionary_direct_layout/metadata.json

@@ -1,6 +1 @@
-{
-  "explain_todo": {
-    "stmt25": true,
-    "stmt26": true
-  }
-}
+{}

parser/testdata/01676_range_hashed_dictionary/metadata.json

@@ -1,5 +1 @@
-{
-  "explain_todo": {
-    "stmt5": true
-  }
-}
+{}

parser/testdata/01681_cache_dictionary_simple_key/metadata.json

@@ -1,5 +1 @@
-{
-  "explain_todo": {
-    "stmt7": true
-  }
-}
+{}

parser/testdata/01760_polygon_dictionaries/metadata.json

@@ -1,5 +1 @@
-{
-  "explain_todo": {
-    "stmt17": true
-  }
-}
+{}

parser/testdata/01765_hashed_dictionary_simple_key/metadata.json

@@ -1,5 +1 @@
-{
-  "explain_todo": {
-    "stmt7": true
-  }
-}
+{}