From 0a83d3cc02c9a55d635052b1a1d45fece0d5e0b2 Mon Sep 17 00:00:00 2001
From: SeaRise <hhssearise@foxmail.com>
Date: Mon, 24 Jul 2023 13:24:51 +0800
Subject: [PATCH] set tiflash pipeline model to true by default
---
sessionctx/variable/session.go | 2 +-
sessionctx/variable/tidb_vars.go | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/sessionctx/variable/session.go b/sessionctx/variable/session.go
index 91206ff8df2c5..0d05480e41ea1 100644
--- a/sessionctx/variable/session.go
+++ b/sessionctx/variable/session.go
@@ -880,7 +880,7 @@ type SessionVars struct {
TiFlashMaxBytesBeforeExternalSort int64
// TiFlashEnablePipelineMode means if we should use pipeline model to execute query or not in tiflash.
- // Default value is `false`, means never use pipeline model in tiflash.
+ // Default value is `true`, means never use pipeline model in tiflash.
// Value set to `true` means try to execute query with pipeline model in tiflash.
TiFlashEnablePipelineMode bool
diff --git a/sessionctx/variable/tidb_vars.go b/sessionctx/variable/tidb_vars.go
index 29e307c520bd8..9d17c85f710ab 100644
--- a/sessionctx/variable/tidb_vars.go
+++ b/sessionctx/variable/tidb_vars.go
@@ -409,7 +409,7 @@ const (
TiDBMaxBytesBeforeTiFlashExternalSort = "tidb_max_bytes_before_tiflash_external_sort"
// TiDBEnableTiFlashPipelineMode means if we should use pipeline model to execute query or not in tiflash.
- // Default value is `false`, means never use pipeline model in tiflash.
+ // Default value is `true`, means never use pipeline model in tiflash.
// Value set to `true` means try to execute query with pipeline model in tiflash.
TiDBEnableTiFlashPipelineMode = "tidb_enable_tiflash_pipeline_model"
@@ -1164,7 +1164,7 @@ const (
DefTiFlashMaxBytesBeforeExternalJoin = -1
DefTiFlashMaxBytesBeforeExternalGroupBy = -1
DefTiFlashMaxBytesBeforeExternalSort = -1
- DefTiDBEnableTiFlashPipelineMode = false
+ DefTiDBEnableTiFlashPipelineMode = true
DefTiDBMPPStoreFailTTL = "60s"
DefTiDBTxnMode = ""
DefTiDBRowFormatV1 = 1