Skip to content

Commit

Permalink
sessionctx: add warning for system variable `tidb_enable_parallel_has…
Browse files Browse the repository at this point in the history
…hagg_spill` (#53917)

close #53919
  • Loading branch information
xzhangxian1008 authored Jun 12, 2024
1 parent 64f5427 commit 8d28b23
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions pkg/sessionctx/variable/sysvar.go
Original file line number Diff line number Diff line change
Expand Up @@ -2381,6 +2381,9 @@ var defaultSysVars = []*SysVar{
{Scope: ScopeGlobal | ScopeSession, Name: TiDBEnableParallelHashaggSpill, Value: BoolToOnOff(DefTiDBEnableParallelHashaggSpill), Type: TypeBool,
SetSession: func(vars *SessionVars, s string) error {
vars.EnableParallelHashaggSpill = TiDBOptOn(s)
if !vars.EnableParallelHashaggSpill {
vars.StmtCtx.AppendWarning(ErrWarnDeprecatedSyntaxSimpleMsg.FastGen("tidb_enable_parallel_hashagg_spill will be removed in the future and hash aggregate spill will be enabled by default."))
}
return nil
},
},
Expand Down

0 comments on commit 8d28b23

Please sign in to comment.