Add EXTERNAL resource pool support for ALTER WORKLOAD GROUP

Support USING EXTERNAL pool_name syntax where the external pool
can appear first in the USING clause, enabling:
- USING EXTERNAL p_ext (external pool only)
- USING EXTERNAL p_ext, p_int (external first, then internal)
- USING p_int, EXTERNAL p_ext (internal first, then external)

Enables AlterWorkloadGroupStatementTests130 and Baselines130.

Author: claude

parser/parse_ddl.go

@@ -6562,15 +6562,29 @@ func (p *Parser) parseAlterWorkloadGroupStatement() (*ast.AlterWorkloadGroupStat
 	// Parse USING clause (resource pool reference)
 	if strings.ToUpper(p.curTok.Literal) == "USING" {
 		p.nextToken() // consume USING
-		stmt.PoolName = p.parseIdentifier()
 
-		// Check for EXTERNAL
-		if p.curTok.Type == TokenComma {
-			p.nextToken()
-		}
+		// Check if EXTERNAL comes first
 		if strings.ToUpper(p.curTok.Literal) == "EXTERNAL" {
 			p.nextToken() // consume EXTERNAL
 			stmt.ExternalPoolName = p.parseIdentifier()
+
+			// Check for comma and internal pool
+			if p.curTok.Type == TokenComma {
+				p.nextToken()
+				stmt.PoolName = p.parseIdentifier()
+			}
+		} else {
+			// Internal pool first
+			stmt.PoolName = p.parseIdentifier()
+
+			// Check for EXTERNAL
+			if p.curTok.Type == TokenComma {
+				p.nextToken()
+			}
+			if strings.ToUpper(p.curTok.Literal) == "EXTERNAL" {
+				p.nextToken() // consume EXTERNAL
+				stmt.ExternalPoolName = p.parseIdentifier()
+			}
 		}
 	}
 

parser/testdata/AlterWorkloadGroupStatementTests130/metadata.json

@@ -1 +1 @@
-{"todo": true}
+{}

parser/testdata/Baselines130_AlterWorkloadGroupStatementTests130/metadata.json

@@ -1 +1 @@
-{"todo": true}
+{}