Many hyperlinks are disabled.
Use anonymous login
to enable hyperlinks.
Overview
Comment: | In the CREATE INDEX statements output by schemalint.tcl, avoid declaring an explicit collation sequence that is the same as the column's default. |
---|---|
Downloads: | Tarball | ZIP archive |
Timelines: | family | ancestors | descendants | both | schemalint |
Files: | files | file ages | folders |
SHA1: |
d3aa067c830e98f2074630c4613c557b |
User & Date: | dan 2015-11-23 18:28:07.584 |
Context
2015-11-30
| ||
18:17 | Fix the schemalint.tcl script to handle identifiers that require quoting. (check-in: 451e0fafbe user: dan tags: schemalint) | |
2015-11-23
| ||
18:28 | In the CREATE INDEX statements output by schemalint.tcl, avoid declaring an explicit collation sequence that is the same as the column's default. (check-in: d3aa067c83 user: dan tags: schemalint) | |
17:14 | Merge latest trunk changes with this branch. (check-in: 8f1ef0904d user: dan tags: schemalint) | |
Changes
Changes to tool/schemalint.tcl.
︙ | ︙ | |||
120 121 122 123 124 125 126 | return $ll } #-------------------------------------------------------------------------- # Formulate a CREATE INDEX statement that creates an index on table $tname. # | | > | < | < < < | < < | > | > > | > > | > > | < < > | 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 | return $ll } #-------------------------------------------------------------------------- # Formulate a CREATE INDEX statement that creates an index on table $tname. # proc eqset_to_index {ctxvar aCollVar tname eqset {range {}}} { upvar $ctxvar G upvar $aCollVar aColl set rangeset [list] foreach e [lsort $eqset] { lappend rangeset [lindex $e 0] [lindex $e 1] ASC } set rangeset [concat $rangeset $range] set lCols [list] set idxname $tname foreach {c collate dir} $rangeset { append idxname "_$c" set coldef $c if {[string compare -nocase $collate $aColl($c)]!=0} { append idxname [string tolower $collate] append coldef " COLLATE $collate" } if {$dir=="DESC"} { append coldef " DESC" append idxname "desc" } lappend lCols $coldef } set create_index "CREATE INDEX $idxname ON ${tname}(" append create_index [join $lCols ", "] append create_index ");" set G(trial.$idxname) $create_index |
︙ | ︙ | |||
179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 | } } set lRet $lNew } } return $lRet } proc find_trial_indexes {ctxvar} { upvar $ctxvar G foreach t $G(trace) { set tname [lindex $t 0] catch { array unset mask } set orderby [list] if {[lindex $t end 0]=="orderby"} { set orderby [lrange [lindex $t end] 1 end] } foreach lCons [expand_or_cons [lrange $t 2 end]] { | > > > > > > > > > > > > > > > > > > | 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 | } } set lRet $lNew } } return $lRet } proc sqlidx_get_coll_map {tname arrayvar} { upvar $arrayvar aColl set colnames [list] db eval "PRAGMA table_info = $tname" x { lappend colnames $x(name) } db eval "CREATE INDEX schemalint_test ON ${tname}([join $colnames ,])" db eval "PRAGMA index_xinfo = schemalint_test" x { set aColl($x(name)) $x(coll) } db eval "DROP INDEX schemalint_test" } proc find_trial_indexes {ctxvar} { upvar $ctxvar G foreach t $G(trace) { set tname [lindex $t 0] catch { array unset mask } # Invoke "PRAGMA table_info" on the table. Use the results to create # an array mapping from column name to collation sequence. Store the # array in local variable aColl. # sqlidx_get_coll_map $tname aColl set orderby [list] if {[lindex $t end 0]=="orderby"} { set orderby [lrange [lindex $t end] 1 end] } foreach lCons [expand_or_cons [lrange $t 2 end]] { |
︙ | ︙ | |||
254 255 256 257 258 259 260 | set ranges [lsort -unique $ranges] if {$orderby != ""} { lappend ranges $orderby } foreach eqset [expand_eq_list [array get eq]] { if {$eqset != ""} { | | | | 273 274 275 276 277 278 279 280 281 282 283 284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301 302 303 | set ranges [lsort -unique $ranges] if {$orderby != ""} { lappend ranges $orderby } foreach eqset [expand_eq_list [array get eq]] { if {$eqset != ""} { eqset_to_index G aColl $tname $eqset } foreach r $ranges { set tail [list] foreach {c collate dir} $r { set bSeen 0 foreach e $eqset { if {[lindex $e 0] == $c} { set bSeen 1 break } } if {$bSeen==0} { lappend tail {*}$r } } if {[llength $tail]} { eqset_to_index G aColl $tname $eqset $r } } } } } } |
︙ | ︙ | |||
357 358 359 360 361 362 363 | sqlidx_one_test 1 { CREATE TABLE t1(a, b, c); CREATE TABLE t2(x, y, z); } { SELECT a FROM t1, t2 WHERE a=? AND x=c } { | | | | | | | | | > > > > > > > > | 376 377 378 379 380 381 382 383 384 385 386 387 388 389 390 391 392 393 394 395 396 397 398 399 400 401 402 403 404 405 406 407 408 409 410 411 412 413 414 415 416 417 418 419 420 421 422 423 424 425 426 427 428 429 430 431 432 433 434 435 436 437 438 439 440 441 442 443 444 445 446 447 | sqlidx_one_test 1 { CREATE TABLE t1(a, b, c); CREATE TABLE t2(x, y, z); } { SELECT a FROM t1, t2 WHERE a=? AND x=c } { {CREATE INDEX t2_x ON t2(x);} {CREATE INDEX t1_a_c ON t1(a, c);} } sqlidx_one_test 2 { CREATE TABLE t1(a, b, c); } { SELECT * FROM t1 WHERE b>?; } { {CREATE INDEX t1_b ON t1(b);} } sqlidx_one_test 3 { CREATE TABLE t1(a, b, c); } { SELECT * FROM t1 WHERE b COLLATE nocase BETWEEN ? AND ? } { {CREATE INDEX t1_bnocase ON t1(b COLLATE NOCASE);} } sqlidx_one_test 4 { CREATE TABLE t1(a, b, c); } { SELECT a FROM t1 ORDER BY b; } { {CREATE INDEX t1_b ON t1(b);} } sqlidx_one_test 5 { CREATE TABLE t1(a, b, c); } { SELECT a FROM t1 WHERE a=? ORDER BY b; } { {CREATE INDEX t1_a_b ON t1(a, b);} } sqlidx_one_test 5 { CREATE TABLE t1(a, b, c); } { SELECT min(a) FROM t1 } { {CREATE INDEX t1_a ON t1(a);} } sqlidx_one_test 6 { CREATE TABLE t1(a, b, c); } { SELECT * FROM t1 ORDER BY a ASC, b COLLATE nocase DESC, c ASC; } { {CREATE INDEX t1_a_bnocasedesc_c ON t1(a, b COLLATE NOCASE DESC, c);} } sqlidx_one_test 7 { CREATE TABLE t1(a COLLATE NOCase, b, c); } { SELECT * FROM t1 WHERE a=? } { {CREATE INDEX t1_a ON t1(a);} } exit } # End of internal test code. #------------------------------------------------------------------------- |
︙ | ︙ |