Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
硕大的木艮
soar
提交
6bb6a2e1
S
soar
项目概览
硕大的木艮
/
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,发现更多精彩内容 >>
未验证
提交
6bb6a2e1
编写于
11月 21, 2018
作者:
martianzhang
提交者:
GitHub
11月 21, 2018
浏览文件
操作
浏览文件
下载
差异文件
Merge pull request #113 from liipx/master
main parser replace vitess to tidb & fix test sqls
上级
6b0730ac
ffa5eea5
变更
5
隐藏空白更改
内联
并排
Showing
5 changed file
with
23 addition
and
17 deletion
+23
-17
advisor/heuristic.go
advisor/heuristic.go
+5
-5
advisor/heuristic_test.go
advisor/heuristic_test.go
+2
-2
advisor/rules.go
advisor/rules.go
+9
-9
cmd/soar/soar.go
cmd/soar/soar.go
+1
-1
doc/FAQ.md
doc/FAQ.md
+6
-0
未找到文件。
advisor/heuristic.go
浏览文件 @
6bb6a2e1
...
@@ -2185,8 +2185,8 @@ func (q *Query4Audit) RuleCompareWithFunction() Rule {
...
@@ -2185,8 +2185,8 @@ func (q *Query4Audit) RuleCompareWithFunction() Rule {
}
}
*/
*/
// func(a) between func(c) and func(d)
case
*
sqlparser
.
RangeCond
:
case
*
sqlparser
.
RangeCond
:
// func(a) between func(c) and func(d)
switch
n
.
Left
.
(
type
)
{
switch
n
.
Left
.
(
type
)
{
case
*
sqlparser
.
SQLVal
,
*
sqlparser
.
ColName
:
case
*
sqlparser
.
SQLVal
,
*
sqlparser
.
ColName
:
default
:
default
:
...
@@ -3231,12 +3231,12 @@ func RuleMySQLError(item string, err error) Rule {
...
@@ -3231,12 +3231,12 @@ func RuleMySQLError(item string, err error) Rule {
ErrString
string
ErrString
string
}
}
//
vitess
语法检查出错返回的是ERR.000
//
tidb parser
语法检查出错返回的是ERR.000
switch
item
{
switch
item
{
case
"ERR.000"
:
case
"ERR.000"
:
return
Rule
{
return
Rule
{
Item
:
item
,
Item
:
item
,
Summary
:
"
MySQL执行出错
"
+
err
.
Error
(),
Summary
:
"
No available MySQL environment, build-in sql parse failed:
"
+
err
.
Error
(),
Severity
:
"L8"
,
Severity
:
"L8"
,
Content
:
err
.
Error
(),
Content
:
err
.
Error
(),
}
}
...
@@ -3270,14 +3270,14 @@ func RuleMySQLError(item string, err error) Rule {
...
@@ -3270,14 +3270,14 @@ func RuleMySQLError(item string, err error) Rule {
case
""
,
"1146"
:
case
""
,
"1146"
:
return
Rule
{
return
Rule
{
Item
:
item
,
Item
:
item
,
Summary
:
"MySQL
执行出错
"
,
Summary
:
"MySQL
execute failed:
"
,
Severity
:
"L0"
,
Severity
:
"L0"
,
Content
:
""
,
Content
:
""
,
}
}
default
:
default
:
return
Rule
{
return
Rule
{
Item
:
item
,
Item
:
item
,
Summary
:
"MySQL
执行出错
"
+
mysqlError
.
ErrString
,
Summary
:
"MySQL
execute failed:
"
+
mysqlError
.
ErrString
,
Severity
:
"L8"
,
Severity
:
"L8"
,
Content
:
mysqlError
.
ErrString
,
Content
:
mysqlError
.
ErrString
,
}
}
...
...
advisor/heuristic_test.go
浏览文件 @
6bb6a2e1
...
@@ -1164,7 +1164,7 @@ func TestRuleRecursiveDependency(t *testing.T) {
...
@@ -1164,7 +1164,7 @@ func TestRuleRecursiveDependency(t *testing.T) {
`ALTER TABLE tbl2 add FOREIGN KEY (p_id) REFERENCES tab1(p_id);`
,
`ALTER TABLE tbl2 add FOREIGN KEY (p_id) REFERENCES tab1(p_id);`
,
},
},
{
{
`ALTER TABLE tbl2 ADD KEY
(p_id) p_id
;`
,
`ALTER TABLE tbl2 ADD KEY
p_id (p_id)
;`
,
},
},
}
}
for
_
,
sql
:=
range
sqls
[
0
]
{
for
_
,
sql
:=
range
sqls
[
0
]
{
...
@@ -2128,7 +2128,7 @@ func TestRuleVarcharVSChar(t *testing.T) {
...
@@ -2128,7 +2128,7 @@ func TestRuleVarcharVSChar(t *testing.T) {
func
TestRuleCreateDualTable
(
t
*
testing
.
T
)
{
func
TestRuleCreateDualTable
(
t
*
testing
.
T
)
{
common
.
Log
.
Debug
(
"Entering function: %s"
,
common
.
GetFunctionName
())
common
.
Log
.
Debug
(
"Entering function: %s"
,
common
.
GetFunctionName
())
sqls
:=
[]
string
{
sqls
:=
[]
string
{
`create table dual(id int, primary key (id));`
,
"create table `dual`(id int, primary key (id));"
,
}
}
for
_
,
sql
:=
range
sqls
{
for
_
,
sql
:=
range
sqls
{
q
,
err
:=
NewQuery4Audit
(
sql
)
q
,
err
:=
NewQuery4Audit
(
sql
)
...
...
advisor/rules.go
浏览文件 @
6bb6a2e1
...
@@ -42,7 +42,7 @@ type Query4Audit struct {
...
@@ -42,7 +42,7 @@ type Query4Audit struct {
// NewQuery4Audit return a struct for Query4Audit
// NewQuery4Audit return a struct for Query4Audit
func
NewQuery4Audit
(
sql
string
,
options
...
string
)
(
*
Query4Audit
,
error
)
{
func
NewQuery4Audit
(
sql
string
,
options
...
string
)
(
*
Query4Audit
,
error
)
{
var
err
,
ti
Err
error
var
err
,
v
Err
error
var
charset
string
var
charset
string
var
collation
string
var
collation
string
...
@@ -55,15 +55,15 @@ func NewQuery4Audit(sql string, options ...string) (*Query4Audit, error) {
...
@@ -55,15 +55,15 @@ func NewQuery4Audit(sql string, options ...string) (*Query4Audit, error) {
}
}
q
:=
&
Query4Audit
{
Query
:
sql
}
q
:=
&
Query4Audit
{
Query
:
sql
}
// vitess语法解析
// vitess 语法解析不上报,以 tidb parser 为主
q
.
Stmt
,
err
=
sqlparser
.
Parse
(
sql
)
q
.
Stmt
,
vErr
=
sqlparser
.
Parse
(
sql
)
if
vErr
!=
nil
{
common
.
Log
.
Warn
(
"NewQuery4Audit vitess parse Error: %s"
,
vErr
.
Error
())
}
// TiDB 语法解析仅作为补充,不检查语法错误
// TODO: charset, collation
// TODO: charset, collation
q
.
TiStmt
,
tiErr
=
ast
.
TiParse
(
sql
,
charset
,
collation
)
// tdib parser 语法解析
if
tiErr
!=
nil
{
q
.
TiStmt
,
err
=
ast
.
TiParse
(
sql
,
charset
,
collation
)
common
.
Log
.
Warn
(
"NewQuery4Audit ast.Tiparse Error: %s"
,
tiErr
.
Error
())
}
return
q
,
err
return
q
,
err
}
}
...
@@ -1205,7 +1205,7 @@ func FormatSuggest(sql string, format string, suggests ...map[string]Rule) (map[
...
@@ -1205,7 +1205,7 @@ func FormatSuggest(sql string, format string, suggests ...map[string]Rule) (map[
}
}
sort
.
Strings
(
sortedMySQLSuggest
)
sort
.
Strings
(
sortedMySQLSuggest
)
if
len
(
sortedMySQLSuggest
)
>
0
{
if
len
(
sortedMySQLSuggest
)
>
0
{
buf
=
append
(
buf
,
"## MySQL
执行出错
\n
"
)
buf
=
append
(
buf
,
"## MySQL
execute failed
\n
"
)
}
}
for
_
,
item
:=
range
sortedMySQLSuggest
{
for
_
,
item
:=
range
sortedMySQLSuggest
{
buf
=
append
(
buf
,
fmt
.
Sprintln
(
suggest
[
item
]
.
Content
))
buf
=
append
(
buf
,
fmt
.
Sprintln
(
suggest
[
item
]
.
Content
))
...
...
cmd/soar/soar.go
浏览文件 @
6bb6a2e1
...
@@ -202,7 +202,7 @@ func main() {
...
@@ -202,7 +202,7 @@ func main() {
if
!
common
.
Config
.
DryRun
{
if
!
common
.
Config
.
DryRun
{
os
.
Exit
(
1
)
os
.
Exit
(
1
)
}
}
//
vitess
语法检查给出的建议 ERR.000
//
tidb parser
语法检查给出的建议 ERR.000
if
common
.
Config
.
TestDSN
.
Disable
{
if
common
.
Config
.
TestDSN
.
Disable
{
mysqlSuggest
[
"ERR.000"
]
=
advisor
.
RuleMySQLError
(
"ERR.000"
,
syntaxErr
)
mysqlSuggest
[
"ERR.000"
]
=
advisor
.
RuleMySQLError
(
"ERR.000"
,
syntaxErr
)
}
}
...
...
doc/FAQ.md
浏览文件 @
6bb6a2e1
...
@@ -9,6 +9,12 @@
...
@@ -9,6 +9,12 @@
*
[
mysql
](
https://www.mysql.com/
)
测试时用来连接测试环境
*
[
mysql
](
https://www.mysql.com/
)
测试时用来连接测试环境
*
[
retool
](
https://github.com/twitchtv/retool
)
: 管理测试开发工具,首次安装耗时会比较长,如:
`gometalinter.v2`
,
`revive`
,
`golangci-lint`
*
[
retool
](
https://github.com/twitchtv/retool
)
: 管理测试开发工具,首次安装耗时会比较长,如:
`gometalinter.v2`
,
`revive`
,
`golangci-lint`
## Web 界面支持
官方不会提供 Web 界面支持,但社区已经有相当多的同学基于 SOAR 开发了衍生的 Web 平台。可以参考如下 ISSUE 的讨论。
> https://github.com/XiaoMi/soar/issues/51
## 命令行参数 `test-dsn`, `online-dsn` 中包含特殊字符怎么办?
## 命令行参数 `test-dsn`, `online-dsn` 中包含特殊字符怎么办?
如果
`test-dsn`
或
`online-dsn`
中包含':', '@', '/', '!'等特殊字符建议在配置文件中配置相关信息,配置文件为YAML格式,需要遵守YAML格式的要求规范。
如果
`test-dsn`
或
`online-dsn`
中包含':', '@', '/', '!'等特殊字符建议在配置文件中配置相关信息,配置文件为YAML格式,需要遵守YAML格式的要求规范。
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录