支持Oracle数据修改备份SQL功能，支持explain,with查询语法,对Oracle表结构和schema显示进行排序,支持sql tuning advisor,Oracle支持执行计划查看，支持Oracle update/insert/delete/create table/create index的语法SQL审核 #701
Conversation
|
你好!感谢你反馈的问题/bug，但是你的描述好像是空的，我们需要你完整的信息，这样才能帮你解决问题 如果不知道怎么写，在新建issue的时候有若干个模板可供选择，祝好！ |
|
1、生成回滚SQL,执行用户需要有grant select any transaction 权限，需要有grant execute on dbms_logmnr 权限 |
|
问题不大, 考虑下我的这几个修改意见, 不要重开pr, 直接在你的分支上提交推送就行. |
|
😅 单元测试也要改一下, 这个比我上面提的要重要一些, 我看还有一些比较简单的引用未声明变量的问题, 一定要修. https://travis-ci.org/github/hhyo/Archery/builds/672116656 看一下这里, 失败的几个都要改到成功, 你本地也可以跑 感谢贡献. |
|
最近 travis-ci 是不是不好使了, 感觉master 上好几次构建都没把状态同步过来, 不行就换 circleci 看看. |
|
这个pr还需要@ericruan-cn 看下，感觉回滚了他前面的提交 |
|
基本都回滚了，建议作者拿到最新更新并解决冲突之后重新提pr
发自我的iPhone
…------------------ 原始邮件 ------------------
发件人: 小圈圈 <notifications@github.com>
发送时间: 2020年4月8日 20:40
收件人: hhyo/Archery <Archery@noreply.github.com>
抄送: ericruan-cn <330518878@qq.com>, Mention <mention@noreply.github.com>
主题: 回复：[hhyo/Archery] 支持Oracle数据修改备份SQL功能，支持explain,with查询语法 (#701)
这个pr还需要@ericruan-cn 看下，感觉回滚了他前面的提交
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
|
这个...你解决一下吧，他的都merge进master了 |
|
最好rebase下 master 然后force push，有一些把之前修改回滚的注意一下。 |
| if close_conn: | ||
| self.close() | ||
| return execute_result | ||
|
|
||
| def backup(self,workflow_id,cursor,begin_time,end_time): |
There was a problem hiding this comment.
几个建议和问题:
- 参数和上面的用法没对上, 这里是必选参数, 上面的是可选参数.
- 这里建议直接传 workflow 对象过去
- 这里的cursor 是不是必须传的, 如果是必须传, 建议改一个有辨识度的名字, 和 backup 的cursor 区分开来, 如果不是必须传, 可以用
self.get_connection get connection然后再拿到cursor
There was a problem hiding this comment.
为了获取Oracle当前的会话信息cursor是必须传的 ，这个cursor就是执行的cursor,必须是同一个会话，才能抓取到回滚会话的信息，当然也可以关闭执行SQL会话，将需要的会话信息查询出来做参数传过去，这样备份需要新开Oracle的连接会话，多一次fork process开销，代码也更麻烦
There was a problem hiding this comment.
参数的问题太专业这边没看懂你指的是什么，一直做 的Oracle运维，没做过研发，抱歉
There was a problem hiding this comment.
是的，这个要想获得备份语句，需要和已经执行的上线sql在同一会话里面，要不然不方便从v$logmnr_contents 获取到undo sql
There was a problem hiding this comment.
def backup(self, workflow, current_cursor, begin_time, end_time):
self.backup(workflow, cursor, begin_time, end_time)
sql/engines/oracle.py
Outdated
| # 创建备份库连接 | ||
| try: | ||
| conn = self.get_backup_connection() | ||
| cur = conn.cursor() |
There was a problem hiding this comment.
这里改成 backup_cursor = conn.cursor() 后面 cur 和 cursor 会看花眼
There was a problem hiding this comment.
本地部署的代码（1.7.8版本）这边加上了Oracle执行计划查看支持（pr已经实现），Oracle数据修改SQL审核（explain plan for 实现），改动的太多了，我按上面要求重新整理测试后再提交一份吧，这个版本上线的Oracle存储过程和本地1.7.8版本的改动不兼容，后续打算把Oracle的sql tuning advisor也集成到SQL优化功能上去，前端代码暂时没看懂，哪位大神能给点思路？
There was a problem hiding this comment.
那块前端代码比较乱，看你需要展示什么信息，如果能做成通用的格式，可以扩充一下现有的engine，具体可以添加一下我的个人微信
There was a problem hiding this comment.
好吧, 下次你提的时候记得在你的fork里新开一个分支, master 我们没法帮你修改, 一般会有保护.
There was a problem hiding this comment.
Oracle的sql tuning advisor是Oracle自带的存储过程，执行普通的查询SQL就能获取到SQL优化报告，展示信息其实就是个查询SQL的返回结果。Oracle执行计划获取和这个思路是一样的，执行Oracle特定的存储过程后返回一个clob类型的查询结果。包括Oracle实例性能诊断优化建议报告(ADDM)，AWR报告，都是可以通过调用Oracle存储过程的形式获取到，如果要丰富Oracle的功能确实需要做个通用格式，能力有限，完全没思路要怎么弄
There was a problem hiding this comment.
oracle要有新功能了，好期待。
我准备五一研究一下archery的源代码，希望能看懂。
|
我把这个pr 的base 改成orable 了, 差不多改好后就merge , merge 完了我帮你整理一下再merge 进 master |
Oracle执行计划查看支持
oracle 新增SQL tunning advisor
添加Oracle sql tuning advisor
oracle sql tuning advisor支持
添加Oracle SQL类型判断
支持Oracle select、update\delete\insert\create table\create index的语法审核。支持数据修改备份功能。支持执行计划查看。修复查询bug
|
改好了，大神们一起看看有啥问题没，sql tuning advisor哪位帮忙优化一下显示效果，有些缩进丢失了，Oracle客户端sqlplus和其它官方工具连接没这个问题 |
|
好期待，对oracle要有增强功能了。感谢jan-song、hhyo和LeoQuote的无私奉献。 |
|
diff的结果有点奇怪，能rebase一下吗？ |
|
@hhyo 我改了base，直接merge进来自己rebase吧。 |
|
你好，我给执行sql的用户授予dba权限，是可以在archery里面看到生产的回滚语句； grant connect,resource to czx; |
|
@czxin788 |
|
感谢，已经验证，想要archery能看到回滚sql，必须开启如下三个权限，缺一不可： |
相关issue #666 #700