Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
m0_67397764
soar
提交
dc1bb566
S
soar
项目概览
m0_67397764
/
soar
与 Fork 源项目一致
Fork自
Xiaomi / soar
通知
1
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
S
soar
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
前往新版Gitcode,体验更适合开发者的 AI 搜索 >>
提交
dc1bb566
编写于
8月 19, 2019
作者:
martianzhang
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
fix multi time `use db` bug
use a; select * from ta; use b; select * from tb;
上级
b4744acf
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
11 addition
and
3 deletion
+11
-3
cmd/soar/soar.go
cmd/soar/soar.go
+11
-2
etc/soar.blacklist
etc/soar.blacklist
+0
-1
未找到文件。
cmd/soar/soar.go
浏览文件 @
dc1bb566
...
...
@@ -175,11 +175,17 @@ func main() {
// 建议去重,减少评审整个文件耗时
// TODO: 由于 a = 11 和 a = '11' 的 fingerprint 相同,这里一旦跳过即无法检查有些建议了,如: ARG.003
if
_
,
ok
:=
suggestMerged
[
id
];
ok
{
continue
// `use ?` 不可以去重,去重后将导致无法切换数据库
if
!
strings
.
HasPrefix
(
fingerprint
,
"use"
)
{
continue
}
}
// 黑名单中的SQL不给建议
if
advisor
.
InBlackList
(
fingerprint
)
{
continue
// `use ?` 不可以出现在黑名单中
if
!
strings
.
HasPrefix
(
fingerprint
,
"use"
)
{
continue
}
}
}
tables
[
id
]
=
ast
.
SchemaMetaInfo
(
sql
,
currentDB
)
...
...
@@ -389,6 +395,9 @@ func main() {
// +++++++++++++++++++++打印单条 SQL 优化建议[开始]++++++++++++++++++++++++++{
common
.
Log
.
Debug
(
"start of print suggestions, Query: %s"
,
q
.
Query
)
if
strings
.
HasPrefix
(
fingerprint
,
"use"
)
{
continue
}
sug
,
str
:=
advisor
.
FormatSuggest
(
q
.
Query
,
currentDB
,
common
.
Config
.
ReportType
,
heuristicSuggest
,
idxSuggest
,
expSuggest
,
proSuggest
,
traceSuggest
,
mysqlSuggest
)
suggestMerged
[
id
]
=
sug
switch
common
.
Config
.
ReportType
{
...
...
etc/soar.blacklist
浏览文件 @
dc1bb566
# 这是一个黑名单例子
## 不评审常见的SET, SHOW, SELECT CONST等完美请求
^use \?$
^set.*
^show.*
^select \?$
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录