move explain code to explain.go

pingcap · Jul 16, 2018 · f0169aa · f0169aa
1 parent 9b201a1
commit f0169aa
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 11 deletions.
diff --git a/executor/builder.go b/executor/builder.go
@@ -618,17 +618,6 @@ func (b *executorBuilder) buildDDL(v *plan.DDL) Executor {
 	return e
 }
 
-func (b *executorBuilder) buildExplain(v *plan.Explain) Executor {
-	e := &ExplainExec{
-		baseExecutor: newBaseExecutor(b.ctx, v.Schema(), v.ExplainID()),
-	}
-	e.rows = make([][]string, 0, len(v.Rows))
-	for _, row := range v.Rows {
-		e.rows = append(e.rows, row)
-	}
-	return e
-}
-
 func (b *executorBuilder) buildUnionScanExec(v *plan.PhysicalUnionScan) Executor {
 	src := b.build(v.Children()[0])
 	if b.err != nil {

diff --git a/executor/explain.go b/executor/explain.go
@@ -15,6 +15,7 @@ package executor
 
 import (
 	"github.com/cznic/mathutil"
+	"github.com/pingcap/tidb/plan"
 	"github.com/pingcap/tidb/util/chunk"
 	"golang.org/x/net/context"
 )
@@ -27,6 +28,19 @@ type ExplainExec struct {
 	cursor int
 }
 
+// buildExplain builds a explain executor. `e.rows` collects final result and
+// displays it in sql-shell.
+func (b *executorBuilder) buildExplain(v *plan.Explain) Executor {
+	e := &ExplainExec{
+		baseExecutor: newBaseExecutor(b.ctx, v.Schema(), v.ExplainID()),
+	}
+	e.rows = make([][]string, 0, len(v.Rows))
+	for _, row := range v.Rows {
+		e.rows = append(e.rows, row)
+	}
+	return e
+}
+
 // Close implements the Executor Close interface.
 func (e *ExplainExec) Close() error {
 	e.rows = nil