Add Location, DataSourceType, PreviousPushDownOption to ALTER EXTERNAL DATA SOURCE

Update AlterExternalDataSourceStatement to output LOCATION as a separate
field and include DataSourceType and PreviousPushDownOption defaults.

Enables: Baselines130_AlterExternalDataSourceStatementTests130,
         AlterExternalDataSourceStatementTests130

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

Author: kyleconroy

ast/external_statements.go

@@ -140,6 +140,9 @@ type ExternalLibraryOption struct {
 // AlterExternalDataSourceStatement represents ALTER EXTERNAL DATA SOURCE statement
 type AlterExternalDataSourceStatement struct {
 	Name                      *Identifier
+	Location                  ScalarExpression
+	DataSourceType            string // HADOOP, etc.
+	PreviousPushDownOption    string // ON, OFF
 	ExternalDataSourceOptions []*ExternalDataSourceLiteralOrIdentifierOption
 }
 

parser/marshal.go

@@ -19589,9 +19589,18 @@ func alterExternalDataSourceStatementToJSON(s *ast.AlterExternalDataSourceStatem
 	node := jsonNode{
 		"$type": "AlterExternalDataSourceStatement",
 	}
+	if s.PreviousPushDownOption != "" {
+		node["PreviousPushDownOption"] = s.PreviousPushDownOption
+	}
 	if s.Name != nil {
 		node["Name"] = identifierToJSON(s.Name)
 	}
+	if s.DataSourceType != "" {
+		node["DataSourceType"] = s.DataSourceType
+	}
+	if s.Location != nil {
+		node["Location"] = scalarExpressionToJSON(s.Location)
+	}
 	if len(s.ExternalDataSourceOptions) > 0 {
 		opts := make([]jsonNode, len(s.ExternalDataSourceOptions))
 		for i, o := range s.ExternalDataSourceOptions {

parser/parse_ddl.go

@@ -9243,7 +9243,10 @@ func (p *Parser) parseAlterExternalDataSourceStatement() (*ast.AlterExternalData
 	}
 	p.nextToken()
 
-	stmt := &ast.AlterExternalDataSourceStatement{}
+	stmt := &ast.AlterExternalDataSourceStatement{
+		DataSourceType:         "HADOOP",
+		PreviousPushDownOption: "ON",
+	}
 
 	// Parse name
 	stmt.Name = p.parseIdentifier()
@@ -9270,6 +9273,20 @@ func (p *Parser) parseAlterExternalDataSourceStatement() (*ast.AlterExternalData
 			p.nextToken()
 		}
 
+		// Handle LOCATION as a separate field
+		if optName == "LOCATION" {
+			if p.curTok.Type == TokenString {
+				strLit, _ := p.parseStringLiteral()
+				stmt.Location = strLit
+			} else {
+				p.nextToken()
+			}
+			if p.curTok.Type == TokenComma {
+				p.nextToken()
+			}
+			continue
+		}
+
 		opt := &ast.ExternalDataSourceLiteralOrIdentifierOption{
 			OptionKind: externalDataSourceOptionKindToPascalCase(optName),
 			Value:      &ast.IdentifierOrValueExpression{},

parser/testdata/AlterExternalDataSourceStatementTests130/metadata.json

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

parser/testdata/Baselines130_AlterExternalDataSourceStatementTests130/metadata.json

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