forked from pingcap/tidb
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add idxadv_test.go for package idxadvisor, need more test cases.
- Loading branch information
1 parent
25b795f
commit 5cc8967
Showing
7 changed files
with
214 additions
and
27 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,132 @@ | ||
| package idxadvisor_test | ||
|
|
||
| import ( | ||
| "encoding/json" | ||
| "fmt" | ||
| "io/ioutil" | ||
| "path/filepath" | ||
| "testing" | ||
|
|
||
| "github.com/juju/errors" | ||
| . "github.com/pingcap/check" | ||
| "github.com/pingcap/tidb/domain" | ||
| idxadv "github.com/pingcap/tidb/idxadvisor" | ||
| "github.com/pingcap/tidb/kv" | ||
| "github.com/pingcap/tidb/session" | ||
| "github.com/pingcap/tidb/sessionctx" | ||
| "github.com/pingcap/tidb/statistics/handle" | ||
| "github.com/pingcap/tidb/store/mockstore" | ||
| "github.com/pingcap/tidb/util/testkit" | ||
| "github.com/pingcap/tidb/util/testleak" | ||
| ) | ||
|
|
||
| var _ = Suite(&testAnalyzeSuite{}) | ||
|
|
||
| type testAnalyzeSuite struct { | ||
| } | ||
|
|
||
| func TestT(t *testing.T) { | ||
| CustomVerboseFlag = true | ||
| TestingT(t) | ||
| } | ||
|
|
||
| func (s *testAnalyzeSuite) loadTableStats(fileName string, dom *domain.Domain) error { | ||
| statsPath := filepath.Join("testdata", fileName) | ||
| bytes, err := ioutil.ReadFile(statsPath) | ||
|
|
||
| if err != nil { | ||
| return err | ||
| } | ||
|
|
||
| statsTbl := &handle.JSONTable{} | ||
| err = json.Unmarshal(bytes, statsTbl) | ||
|
|
||
| if err != nil { | ||
| return err | ||
| } | ||
|
|
||
| statsHandle := dom.StatsHandle() | ||
| err = statsHandle.LoadStatsFromJSON(dom.InfoSchema(), statsTbl) | ||
|
|
||
| if err != nil { | ||
| return err | ||
| } | ||
|
|
||
| return nil | ||
| } | ||
|
|
||
| func (s *testAnalyzeSuite) TestSQLClient(c *C) { | ||
| err := idxadv.RunSqlClient("test-mode") | ||
| c.Assert(err, IsNil) | ||
| } | ||
|
|
||
| func (s *testAnalyzeSuite) TestIndexAdvisor(c *C) { | ||
| defer testleak.AfterTest(c)() | ||
| store, dom, err := newStoreWithBootstrap() | ||
| c.Assert(err, IsNil) | ||
|
|
||
| testkit := testkit.NewTestKit(c, store) | ||
| idxadv.MockNewIdxAdv() | ||
| defer func() { | ||
| dom.Close() | ||
| store.Close() | ||
| }() | ||
|
|
||
| testkit.MustExec("use test") | ||
| testkit.MustExec("drop table if exists t, t1") | ||
| testkit.MustExec("create table t (a int primary key, b int, c varchar(200), d datetime DEFAULT CURRENT_TIMESTAMP, e int, ts timestamp DEFAULT CURRENT_TIMESTAMP)") | ||
| testkit.MustExec("create table t1 (a int, b int)") | ||
|
|
||
| err = s.loadTableStats("analyzesSuiteTestIndexReadT.json", dom) | ||
| c.Assert(err, IsNil) | ||
|
|
||
| for i := 1; i < 16; i++ { | ||
| testkit.MustExec(fmt.Sprintf("insert into t1 values(%v, %v)", i, i)) | ||
| } | ||
| testkit.MustExec("analyze table t1") | ||
|
|
||
| ctx := testkit.Se.(sessionctx.Context) | ||
| sessionVars := ctx.GetSessionVars() | ||
| sessionVars.HashAggFinalConcurrency = 1 | ||
| sessionVars.HashAggPartialConcurrency = 1 | ||
| connID := sessionVars.ConnectionID | ||
|
|
||
| tests := []struct { | ||
| sql []string | ||
| res string | ||
| }{ | ||
| { | ||
| sql: []string{"select count(*) from t group by e", | ||
| "select * from t order by e limit 10"}, | ||
| res: "t: (e)", | ||
| }, | ||
| } | ||
|
|
||
| testkit.MustExec("set tidb_enable_index_advisor=1") | ||
| for _, tt := range tests { | ||
| for _, sql := range tt.sql { | ||
| testkit.Exec(sql) | ||
| } | ||
| res, err := idxadv.GetRecommendIdxStr(connID) | ||
| c.Assert(err, IsNil) | ||
| c.Assert(res, Equals, tt.res, Commentf("for %v", tt.sql)) | ||
| } | ||
| } | ||
|
|
||
| func newStoreWithBootstrap() (kv.Storage, *domain.Domain, error) { | ||
| store, err := mockstore.NewMockTikvStore() | ||
| if err != nil { | ||
| return nil, nil, errors.Trace(err) | ||
| } | ||
|
|
||
| session.SetSchemaLease(0) | ||
| session.DisableStats4Test() | ||
|
|
||
| dom, err := session.BootstrapSession(store) | ||
| if err != nil { | ||
| return nil, nil, err | ||
| } | ||
|
|
||
| dom.SetStatsUpdating(true) | ||
| return store, dom, errors.Trace(err) | ||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.