Rename UpdateItem*

Author: apstndb

ast/ast.go

@@ -544,8 +544,8 @@ type UpdateItem interface {
 	isUpdateItem()
 }
 
-func (UpdateItemAssign) isUpdateItem() {}
-func (UpdateItemNested) isUpdateItem() {}
+func (UpdateItemSetValue) isUpdateItem() {}
+func (UpdateItemDML) isUpdateItem()      {}
 
 // ChangeStreamFor represents FOR clause in CREATE/ALTER CHANGE STREAM statement.
 type ChangeStreamFor interface {
@@ -4063,21 +4063,21 @@ type Update struct {
 	ThenReturn *ThenReturn // optional
 }
 
-// UpdateItemNested is nested update node in UPDATE statement.
+// UpdateItemDML is nested update UpdateItem node in UPDATE statement.
 //
 //	({{.DML | sql}})
-type UpdateItemNested struct {
+type UpdateItemDML struct {
 	// pos = Lparen
 	// end = Rparen + 1
 
 	Lparen, Rparen token.Pos // position of "(", ")"
 	DML            DML
 }
 
-// UpdateItemAssign is assignment node in UPDATE statement .
+// UpdateItemSetValue is assignment style UpdateItem node in UPDATE statement .
 //
 //	{{.Path | sqlJoin "."}} = {{.DefaultExpr | sql}}
-type UpdateItemAssign struct {
+type UpdateItemSetValue struct {
 	// pos = Path[0].pos
 	// end = DefaultExpr.end
 

ast/pos.go

@@ -1351,11 +1351,11 @@ func (u *Update) SQL() string {
 		sqlOpt(" ", u.ThenReturn, "")
 }
 
-func (u *UpdateItemAssign) SQL() string {
+func (u *UpdateItemSetValue) SQL() string {
 	return sqlJoin(u.Path, ".") + " = " + u.DefaultExpr.SQL()
 }
 
-func (u *UpdateItemNested) SQL() string {
+func (u *UpdateItemDML) SQL() string {
 	return "(" + u.DML.SQL() + ")"
 }
 

ast/sql.go

@@ -5204,7 +5204,7 @@ func (p *Parser) parseUpdateItem() ast.UpdateItem {
 		lparen := p.expect("(").Pos
 		dml := p.parseDML(true)
 		rparen := p.expect(")").Pos
-		return &ast.UpdateItemNested{
+		return &ast.UpdateItemDML{
 			Lparen: lparen,
 			Rparen: rparen,
 			DML:    dml,
@@ -5215,7 +5215,7 @@ func (p *Parser) parseUpdateItem() ast.UpdateItem {
 	p.expect("=")
 	defaultExpr := p.parseDefaultExpr()
 
-	return &ast.UpdateItemAssign{
+	return &ast.UpdateItemSetValue{
 		Path:        path,
 		DefaultExpr: defaultExpr,
 	}

parser.go

@@ -25,7 +25,7 @@ WHERE SingerId = 3 AND s.Albums.title = 'Go! Go! Go!'
     },
   },
   Updates: []ast.UpdateItem{
-    &ast.UpdateItemNested{
+    &ast.UpdateItemDML{
       Lparen: 23,
       Rparen: 85,
       DML:    &ast.Delete{
@@ -84,7 +84,7 @@ WHERE SingerId = 3 AND s.Albums.title = 'Go! Go! Go!'
         },
       },
     },
-    &ast.UpdateItemNested{
+    &ast.UpdateItemDML{
       Lparen: 90,
       Rparen: 204,
       DML:    &ast.Update{
@@ -117,7 +117,7 @@ WHERE SingerId = 3 AND s.Albums.title = 'Go! Go! Go!'
           },
         },
         Updates: []ast.UpdateItem{
-          &ast.UpdateItemAssign{
+          &ast.UpdateItemSetValue{
             Path: []*ast.Ident{
               &ast.Ident{
                 NamePos: 121,
@@ -167,7 +167,7 @@ WHERE SingerId = 3 AND s.Albums.title = 'Go! Go! Go!'
         },
       },
     },
-    &ast.UpdateItemNested{
+    &ast.UpdateItemDML{
       Lparen: 209,
       Rparen: 275,
       DML:    &ast.Insert{

testdata/result/dml/nested_update_delete_update_insert.sql.txt

@@ -23,7 +23,7 @@ WHERE s.SingerId = 5 AND s.AlbumInfo.title = "Fire is Hot"
     },
   },
   Updates: []ast.UpdateItem{
-    &ast.UpdateItemNested{
+    &ast.UpdateItemDML{
       Lparen: 21,
       Rparen: 105,
       DML:    &ast.Insert{

testdata/result/dml/nested_update_insert_song.sql.txt

@@ -24,7 +24,7 @@ WHERE s.SingerId = 5 and s.AlbumInfo.title = "Fire is Hot"
     },
   },
   Updates: []ast.UpdateItem{
-    &ast.UpdateItemNested{
+    &ast.UpdateItemDML{
       Lparen: 21,
       Rparen: 102,
       DML:    &ast.Insert{
@@ -69,7 +69,7 @@ WHERE s.SingerId = 5 and s.AlbumInfo.title = "Fire is Hot"
         },
       },
     },
-    &ast.UpdateItemAssign{
+    &ast.UpdateItemSetValue{
       Path: []*ast.Ident{
         &ast.Ident{
           NamePos: 109,

testdata/result/dml/nested_update_insert_song_path.sql.txt

@@ -23,7 +23,7 @@ WHERE s.SingerId = 5 AND s.AlbumInfo.title = "Fire is Hot"
     },
   },
   Updates: []ast.UpdateItem{
-    &ast.UpdateItemNested{
+    &ast.UpdateItemDML{
       Lparen: 21,
       Rparen: 73,
       DML:    &ast.Insert{

testdata/result/dml/nested_update_insert_string.sql.txt

@@ -25,7 +25,7 @@ WHERE s.SingerId = 5
     },
   },
   Updates: []ast.UpdateItem{
-    &ast.UpdateItemNested{
+    &ast.UpdateItemDML{
       Lparen: 21,
       Rparen: 175,
       DML:    &ast.Update{
@@ -58,7 +58,7 @@ WHERE s.SingerId = 5
           },
         },
         Updates: []ast.UpdateItem{
-          &ast.UpdateItemNested{
+          &ast.UpdateItemDML{
             Lparen: 57,
             Rparen: 135,
             DML:    &ast.Insert{

testdata/result/dml/nested_update_update_nested_insert.sql.txt

@@ -26,7 +26,7 @@ WHERE s.SingerId = 5
     },
   },
   Updates: []ast.UpdateItem{
-    &ast.UpdateItemNested{
+    &ast.UpdateItemDML{
       Lparen: 21,
       Rparen: 189,
       DML:    &ast.Update{
@@ -59,7 +59,7 @@ WHERE s.SingerId = 5
           },
         },
         Updates: []ast.UpdateItem{
-          &ast.UpdateItemNested{
+          &ast.UpdateItemDML{
             Lparen: 58,
             Rparen: 148,
             DML:    &ast.Update{
@@ -87,7 +87,7 @@ WHERE s.SingerId = 5
                 },
               },
               Updates: []ast.UpdateItem{
-                &ast.UpdateItemAssign{
+                &ast.UpdateItemSetValue{
                   Path: []*ast.Ident{
                     &ast.Ident{
                       NamePos: 91,