clean up

Author: angelamayxie

sql/analyzer/triggers.go

@@ -410,7 +410,6 @@ func applyTrigger(ctx *sql.Context, a *Analyzer, originalNode, n sql.Node, scope
 				node, err := n.WithChildren(triggerExecutor)
 				return node, transform.NewTree, err
 			} else {
-				// TODO: add HasAfterTrigger flag for Update node
 				return plan.NewTriggerExecutor(n, triggerLogic, plan.UpdateTrigger, plan.TriggerTime(trigger.TriggerTime), sql.TriggerDefinition{
 					Name:            trigger.TriggerName,
 					CreateStatement: trigger.CreateTriggerString,
@@ -440,7 +439,7 @@ func applyTrigger(ctx *sql.Context, a *Analyzer, originalNode, n sql.Node, scope
 				node, err := n.WithChildren(triggerExecutor)
 				return node, transform.NewTree, err
 			} else {
-				// TODO: add HasAfterTrigger flag for DeleteFrom node
+				// TODO: add HasAfterTrigger flag for DeleteFrom node once DELETE...RETURNING has been implemented
 				return plan.NewTriggerExecutor(n, triggerLogic, plan.DeleteTrigger, plan.TriggerTime(trigger.TriggerTime), sql.TriggerDefinition{
 					Name:            trigger.TriggerName,
 					CreateStatement: trigger.CreateTriggerString,

sql/rowexec/dml_iters.go

@@ -299,6 +299,8 @@ func (t *triggerIter) Next(ctx *sql.Context) (row sql.Row, returnErr error) {
 
 func (t *triggerIter) getReturningRow(ctx *sql.Context, row sql.Row) (sql.Row, error, bool) {
 	if tableEditor, isTableEditor := t.child.(*plan.TableEditorIter); isTableEditor {
+		// TODO: get returning rows for REPLACE and DELETE once REPLACE...RETURNING and DELETE...RETURNING have been
+		//  implemented
 		if insert, isInsert := tableEditor.InnerIter().(*insertIter); isInsert {
 			if len(insert.returnExprs) > 0 {
 				retRow, err := insert.getReturningRow(ctx, row)

sql/rowexec/insert.go

@@ -364,9 +364,6 @@ func (i *insertIter) updateLastInsertId(ctx *sql.Context, row sql.Row) {
 func (i *insertIter) getAutoIncVal(row sql.Row) int64 {
 	for i, expr := range i.insertExprs {
 		if _, ok := expr.(*expression.AutoIncrement); ok {
-			if len(row) == 1 && i == 1 {
-				fmt.Println("here")
-			}
 			return toInt64(row[i])
 		}
 	}