Skip to content

Commit

Permalink
Comment out unused code
Browse files Browse the repository at this point in the history
  • Loading branch information
zachmu committed Jan 8, 2025
1 parent 0c2d5a9 commit 6b52b39
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions server/ast/alter_table.go
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,12 @@ func nodeAlterTable(ctx *Context, node *tree.AlterTable) (vitess.Statement, erro
}

// Otherwise emit warnings now, then return an AlterTable statement
//lint:ignore SA9003 For future use
if len(noOps) > 0 {
// TODO: we don't have a way to send or store the warnings alongside a valid AlterTable statement. We could either
// get a *sql.Context here and emit warnings, or we could store the warnings in the Context and make the caller
// emit them before it sends |ReadyForQuery|
}
// TODO: we don't have a way to send or store the warnings alongside a valid AlterTable statement. We could either
// get a *sql.Context here and emit warnings, or we could store the warnings in the Context and make the caller
// emit them before it sends |ReadyForQuery|
// if len(noOps) > 0 {
// emit warnings
// }

return &vitess.AlterTable{
Table: tableName,
Expand Down

0 comments on commit 6b52b39

Please sign in to comment.