From 60fe0441a47a9e2ecde18a7fba5b9ce67fa50d36 Mon Sep 17 00:00:00 2001 From: bb7133 Date: Thu, 18 Apr 2019 09:53:36 +0800 Subject: [PATCH 1/2] executor, infoschema: `SHOW COLLATIONS` shows supported collations only --- executor/show.go | 6 +++--- executor/show_test.go | 2 +- go.mod | 2 ++ infoschema/tables.go | 6 +++--- infoschema/tables_test.go | 2 +- 5 files changed, 10 insertions(+), 8 deletions(-) diff --git a/executor/show.go b/executor/show.go index fb5dee5b70e41..81736cc272b37 100644 --- a/executor/show.go +++ b/executor/show.go @@ -507,7 +507,7 @@ func (e *ShowExec) fetchShowIndex() error { // fetchShowCharset gets all charset information and fill them into e.rows. // See http://dev.mysql.com/doc/refman/5.7/en/show-character-set.html func (e *ShowExec) fetchShowCharset() error { - descs := charset.GetAllCharsets() + descs := charset.GetSupportedCharsets() for _, desc := range descs { e.appendRow([]interface{}{ desc.Name, @@ -594,7 +594,7 @@ func (e *ShowExec) fetchShowStatus() error { } func getDefaultCollate(charsetName string) string { - for _, c := range charset.GetAllCharsets() { + for _, c := range charset.GetSupportedCharsets() { if strings.EqualFold(c.Name, charsetName) { return c.DefaultCollation } @@ -897,7 +897,7 @@ func (e *ShowExec) fetchShowCreateDatabase() error { } func (e *ShowExec) fetchShowCollation() error { - collations := charset.GetCollations() + collations := charset.GetSupportedCollations() for _, v := range collations { isDefault := "" if v.IsDefault { diff --git a/executor/show_test.go b/executor/show_test.go index bd623916447bd..018d42d6abaaf 100644 --- a/executor/show_test.go +++ b/executor/show_test.go @@ -239,7 +239,7 @@ func (s *testSuite2) TestShow2(c *C) { tk.MustQuery(`describe t`).Check(testutil.RowsWithSep(",", "c,int(11),YES,,,")) tk.MustQuery(`show columns from v`).Check(testutil.RowsWithSep(",", "c,int(11),YES,,,")) tk.MustQuery(`describe v`).Check(testutil.RowsWithSep(",", "c,int(11),YES,,,")) - tk.MustQuery("show collation where Charset = 'utf8' and Collation = 'utf8_bin'").Check(testutil.RowsWithSep(",", "utf8_bin,utf8,83,,Yes,1")) + tk.MustQuery("show collation where Charset = 'utf8' and Collation = 'utf8_bin'").Check(testutil.RowsWithSep(",", "utf8_bin,utf8,83,Yes,Yes,1")) tk.MustQuery("show tables").Check(testkit.Rows("t", "v")) tk.MustQuery("show full tables").Check(testkit.Rows("t BASE TABLE", "v VIEW")) ctx := tk.Se.(sessionctx.Context) diff --git a/go.mod b/go.mod index 9ac0eaa65e863..11a55b19b4549 100644 --- a/go.mod +++ b/go.mod @@ -90,3 +90,5 @@ require ( sourcegraph.com/sourcegraph/appdash v0.0.0-20180531100431-4c381bd170b4 sourcegraph.com/sourcegraph/appdash-data v0.0.0-20151005221446-73f23eafcf67 ) + +replace github.com/pingcap/parser => github.com/bb7133/parser v0.0.0-20190418030907-c05371091476 diff --git a/infoschema/tables.go b/infoschema/tables.go index 4a848f64ba828..acbb0bbcc8fbc 100644 --- a/infoschema/tables.go +++ b/infoschema/tables.go @@ -563,7 +563,7 @@ var tableTiDBHotRegionsCols = []columnInfo{ func dataForCharacterSets() (records [][]types.Datum) { - charsets := charset.GetAllCharsets() + charsets := charset.GetSupportedCharsets() for _, charset := range charsets { @@ -579,7 +579,7 @@ func dataForCharacterSets() (records [][]types.Datum) { func dataForCollations() (records [][]types.Datum) { - collations := charset.GetCollations() + collations := charset.GetSupportedCollations() for _, collation := range collations { @@ -600,7 +600,7 @@ func dataForCollations() (records [][]types.Datum) { func dataForCollationCharacterSetApplicability() (records [][]types.Datum) { - collations := charset.GetCollations() + collations := charset.GetSupportedCollations() for _, collation := range collations { diff --git a/infoschema/tables_test.go b/infoschema/tables_test.go index a6f229bd32f66..e79d8a87a8424 100644 --- a/infoschema/tables_test.go +++ b/infoschema/tables_test.go @@ -168,7 +168,7 @@ func (s *testTableSuite) TestCharacterSetCollations(c *C) { // The is_default column is not important // but the id's are used by client libraries and must be stable tk.MustQuery("SELECT character_set_name, id, sortlen FROM information_schema.collations ORDER BY collation_name").Check( - testkit.Rows("armscii8 64 1", "armscii8 32 1", "ascii 65 1", "ascii 11 1", "big5 84 1", "big5 1 1", "binary 63 1", "cp1250 66 1", "cp1250 44 1", "cp1250 34 1", "cp1250 26 1", "cp1250 99 1", "cp1251 50 1", "cp1251 14 1", "cp1251 51 1", "cp1251 52 1", "cp1251 23 1", "cp1256 67 1", "cp1256 57 1", "cp1257 58 1", "cp1257 59 1", "cp1257 29 1", "cp850 80 1", "cp850 4 1", "cp852 81 1", "cp852 40 1", "cp866 68 1", "cp866 36 1", "cp932 96 1", "cp932 95 1", "dec8 69 1", "dec8 3 1", "eucjpms 98 1", "eucjpms 97 1", "euckr 85 1", "euckr 19 1", "gb2312 86 1", "gb2312 24 1", "gbk 87 1", "gbk 28 1", "geostd8 93 1", "geostd8 92 1", "greek 70 1", "greek 25 1", "hebrew 71 1", "hebrew 16 1", "hp8 72 1", "hp8 6 1", "keybcs2 73 1", "keybcs2 37 1", "koi8r 74 1", "koi8r 7 1", "koi8u 75 1", "koi8u 22 1", "latin1 47 1", "latin1 15 1", "latin1 48 1", "latin1 49 1", "latin1 5 1", "latin1 31 1", "latin1 94 1", "latin1 8 1", "latin2 77 1", "latin2 27 1", "latin2 2 1", "latin2 9 1", "latin2 21 1", "latin5 78 1", "latin5 30 1", "latin7 79 1", "latin7 20 1", "latin7 41 1", "latin7 42 1", "macce 43 1", "macce 38 1", "macroman 53 1", "macroman 39 1", "sjis 88 1", "sjis 13 1", "swe7 82 1", "swe7 10 1", "tis620 89 1", "tis620 18 1", "ucs2 90 1", "ucs2 149 1", "ucs2 138 1", "ucs2 139 1", "ucs2 145 1", "ucs2 134 1", "ucs2 35 1", "ucs2 159 1", "ucs2 148 1", "ucs2 146 1", "ucs2 129 1", "ucs2 130 1", "ucs2 140 1", "ucs2 144 1", "ucs2 133 1", "ucs2 143 1", "ucs2 131 1", "ucs2 147 1", "ucs2 141 1", "ucs2 132 1", "ucs2 142 1", "ucs2 135 1", "ucs2 136 1", "ucs2 137 1", "ucs2 150 1", "ucs2 128 1", "ucs2 151 1", "ujis 91 1", "ujis 12 1", "utf16 55 1", "utf16 122 1", "utf16 111 1", "utf16 112 1", "utf16 118 1", "utf16 107 1", "utf16 54 1", "utf16 121 1", "utf16 119 1", "utf16 102 1", "utf16 103 1", "utf16 113 1", "utf16 117 1", "utf16 106 1", "utf16 116 1", "utf16 104 1", "utf16 120 1", "utf16 114 1", "utf16 105 1", "utf16 115 1", "utf16 108 1", "utf16 109 1", "utf16 110 1", "utf16 123 1", "utf16 101 1", "utf16 124 1", "utf16le 62 1", "utf16le 56 1", "utf32 61 1", "utf32 181 1", "utf32 170 1", "utf32 171 1", "utf32 177 1", "utf32 166 1", "utf32 60 1", "utf32 180 1", "utf32 178 1", "utf32 161 1", "utf32 162 1", "utf32 172 1", "utf32 176 1", "utf32 165 1", "utf32 175 1", "utf32 163 1", "utf32 179 1", "utf32 173 1", "utf32 164 1", "utf32 174 1", "utf32 167 1", "utf32 168 1", "utf32 169 1", "utf32 182 1", "utf32 160 1", "utf32 183 1", "utf8 83 1", "utf8 213 1", "utf8 202 1", "utf8 203 1", "utf8 209 1", "utf8 198 1", "utf8 33 1", "utf8 223 1", "utf8 212 1", "utf8 210 1", "utf8 193 1", "utf8 194 1", "utf8 204 1", "utf8 208 1", "utf8 197 1", "utf8 207 1", "utf8 195 1", "utf8 211 1", "utf8 205 1", "utf8 196 1", "utf8 206 1", "utf8 199 1", "utf8 200 1", "utf8 201 1", "utf8 214 1", "utf8 192 1", "utf8 215 1", "utf8mb4 255 1", "utf8mb4 46 1", "utf8mb4 245 1", "utf8mb4 234 1", "utf8mb4 235 1", "utf8mb4 241 1", "utf8mb4 230 1", "utf8mb4 45 1", "utf8mb4 244 1", "utf8mb4 242 1", "utf8mb4 225 1", "utf8mb4 226 1", "utf8mb4 236 1", "utf8mb4 240 1", "utf8mb4 229 1", "utf8mb4 239 1", "utf8mb4 227 1", "utf8mb4 243 1", "utf8mb4 237 1", "utf8mb4 228 1", "utf8mb4 238 1", "utf8mb4 231 1", "utf8mb4 232 1", "utf8mb4 233 1", "utf8mb4 246 1", "utf8mb4 224 1", "utf8mb4 247 1")) + testkit.Rows("ascii 65 1", "binary 63 1", "latin1 47 1", "utf8 83 1", "utf8mb4 46 1")) // Test charset/collation in information_schema.COLUMNS table. tk.MustExec("DROP DATABASE IF EXISTS charset_collate_test") From 3c8bbf008cba61ded355588b81eb333cc0e275aa Mon Sep 17 00:00:00 2001 From: bb7133 Date: Wed, 24 Apr 2019 11:19:40 +0800 Subject: [PATCH 2/2] update go.mod / go.sum --- go.mod | 4 +--- go.sum | 4 ++-- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/go.mod b/go.mod index 11a55b19b4549..f6055ae5e1455 100644 --- a/go.mod +++ b/go.mod @@ -51,7 +51,7 @@ require ( github.com/pingcap/goleveldb v0.0.0-20171020122428-b9ff6c35079e github.com/pingcap/kvproto v0.0.0-20190215154024-7f2fc73ef562 github.com/pingcap/log v0.0.0-20190307075452-bd41d9273596 - github.com/pingcap/parser v0.0.0-20190418074231-2cdba8195b40 + github.com/pingcap/parser v0.0.0-20190424024541-e2cdb851bce2 github.com/pingcap/pd v2.1.0-rc.4+incompatible github.com/pingcap/tidb-tools v2.1.3-0.20190321065848-1e8b48f5c168+incompatible github.com/pingcap/tipb v0.0.0-20190107072121-abbec73437b7 @@ -90,5 +90,3 @@ require ( sourcegraph.com/sourcegraph/appdash v0.0.0-20180531100431-4c381bd170b4 sourcegraph.com/sourcegraph/appdash-data v0.0.0-20151005221446-73f23eafcf67 ) - -replace github.com/pingcap/parser => github.com/bb7133/parser v0.0.0-20190418030907-c05371091476 diff --git a/go.sum b/go.sum index a284ae14f56b8..9328cfd82a629 100644 --- a/go.sum +++ b/go.sum @@ -124,8 +124,8 @@ github.com/pingcap/kvproto v0.0.0-20190215154024-7f2fc73ef562 h1:32oF1/8lVnBR2JV github.com/pingcap/kvproto v0.0.0-20190215154024-7f2fc73ef562/go.mod h1:QMdbTAXCHzzygQzqcG9uVUgU2fKeSN1GmfMiykdSzzY= github.com/pingcap/log v0.0.0-20190307075452-bd41d9273596 h1:t2OQTpPJnrPDGlvA+3FwJptMTt6MEPdzK1Wt99oaefQ= github.com/pingcap/log v0.0.0-20190307075452-bd41d9273596/go.mod h1:WpHUKhNZ18v116SvGrmjkA9CBhYmuUTKL+p8JC9ANEw= -github.com/pingcap/parser v0.0.0-20190418074231-2cdba8195b40 h1:fw59RpNk5buTOZUGtv5X2vJzgbF9yoJ/I1kEUFmgbt0= -github.com/pingcap/parser v0.0.0-20190418074231-2cdba8195b40/go.mod h1:1FNvfp9+J0wvc4kl8eGNh7Rqrxveg15jJoWo/a0uHwA= +github.com/pingcap/parser v0.0.0-20190424024541-e2cdb851bce2 h1:WZPcqMEpW1PFsusiCEiNlatZdI8dGURXneIUrFNmehI= +github.com/pingcap/parser v0.0.0-20190424024541-e2cdb851bce2/go.mod h1:1FNvfp9+J0wvc4kl8eGNh7Rqrxveg15jJoWo/a0uHwA= github.com/pingcap/pd v2.1.0-rc.4+incompatible h1:/buwGk04aHO5odk/+O8ZOXGs4qkUjYTJ2UpCJXna8NE= github.com/pingcap/pd v2.1.0-rc.4+incompatible/go.mod h1:nD3+EoYes4+aNNODO99ES59V83MZSI+dFbhyr667a0E= github.com/pingcap/tidb-tools v2.1.3-0.20190321065848-1e8b48f5c168+incompatible h1:MkWCxgZpJBgY2f4HtwWMMFzSBb3+JPzeJgF3VrXE/bU=