登录
注册
开源
企业版
高校版
搜索
帮助中心
使用条款
关于我们
开源
企业版
高校版
私有云
模力方舟
AI 队友
登录
注册
代码拉取完成,页面将自动刷新
捐赠
捐赠前请先登录
取消
前往登录
扫描微信二维码支付
取消
支付完成
支付提示
将跳转至支付宝完成支付
确定
取消
Watch
不关注
关注所有动态
仅关注版本发行动态
关注但不提醒动态
31
Star
32
Fork
225
openGauss
/
Plugin
代码
Issues
50
Pull Requests
1
Wiki
统计
流水线
服务
JavaDoc
PHPDoc
质量分析
Jenkins for Gitee
腾讯云托管
腾讯云 Serverless
悬镜安全
阿里云 SAE
Codeblitz
SBOM
我知道了,不再自动展开
1417
修复:B兼容触发器语法问题、show问题
已合并
chenbd:fix330
openGauss:master
chenbd
创建于 2024-03-08 11:23
克隆/下载
HTTPS
SSH
复制
下载 Email Patch
下载 Diff 文件
<!-- 感谢您提交Pull Reqeust --> <!-- 提交说明: 请按照以下的模板提交PR,下列内容均为必填项。如果未补充对应内容,不允许提交PR。 --> 【标题】(请简要描述下实现的内容) 【实现内容】: 修复 B 兼容触发器语法问题、show问题 【根因分析】: 触发器B格式语法实现不合理。 【实现方案】: ``` 语法问题是因为触发器中语法 若subprogram body 前面的部分规约规则的语法内容为空时,会提前读取Begin 以判断是否要进行规约操作。 而原始设置可以被解析为BEGIN_B_BLOCK 的地方,这里如果前面是空的,那么会已经读取了begin。那么会导致KW_MASK 的session变量设置后,不会再解析Begin关键字,也就无法映射到 BEGIN_B_BLOCK 从而导致使用subprogram body 中的解析规则。 修复方案为: 提前设置 B_KWMASK_CREATE_TRIGGER 的标记,这时候已经能确认是CREATE TRIGGER 语法了。原始需求PR https://gitee.com/opengauss/Plugin/pulls/575 show 问题的修复方案是去掉b_cmpt_mode 的参数对此语法的影响 曾经修复过类似问题 https://gitee.com/opengauss/openGauss-server/pulls/3213/ 去掉的原理是,我们设计了 B模式 PLSQL 的语法形式创建过程、函数、触发器的原则是 用delimiter // create xxxx begin xxx end// delimiter ; 这种形式创建的,还要按照 这种形式dump 用 create xxx declare begin xxx end;/ 的方式创建的,还要按照这种形式dump 而且对过程和函数都是这么个处理逻辑,不应该和b_cmpt_mode 再度挂钩。 历史开启b_cmpt_mode方式创建的触发器,dump的时候 不会带有delimiter 信息,导入新版本也能保证导入导出正常。 因为新版本不受到b_cmpt_mode影响 满足前向兼容,如下图 ```  【关联需求或issue】: https://e.gitee.com/opengaussorg/issues/table?issue=I92PY4 【测试类型:SQL功能】【测试版本:6.0.0】show create trigger得到的SQL无法再次执行 https://e.gitee.com/opengaussorg/dashboard?issue=I94H3U 【开发自验报告】: 1. 请附上自验结果(内容或者截图) 问题1 已经补充在fastcheck 问题2 验证默认开启b cmptmode 导入导出正常。 关闭也正常 中途开启、关闭也可以。 通过alter database set 的方式设置参数默认值,默认开启时:  导入导出正常   导出内容如下  2. 是否可以添加fastcheck测试用例,如是,请补充fastcheck用例  已添加 3. 是否涉及资料修改,如是,在docs仓库补充资料 4. 是否考虑支撑升级和在线扩容等扩展场景 5. 是否考虑异常场景/并发场景/前向兼容/性能场景 6. 是否对其他模块产生影响 以上不涉及 【其他说明】:
怎样手动合并此 Pull Request
git checkout master
git pull https://gitee.com/cbd123cbd/Plugin.git fix330
git push origin master
评论
8
提交
1
文件
3
检查
代码问题
0
批量操作
展开设置
折叠设置
审查
Code Owner
审查人员
吴岳川
wuyuechuan
chenxiaobin
chenxiaobin19
pengjiong
totaj
熊小军
xiong_xjun
wumingyuan
wumingyuan
陈栋
chendong76
zhaosiqi
dreamalloc
jeremytim
jeremytim
胡正超
gentle_hu
zhaojun
kelayzhao
huyinghao
yinghaohu
zhangxubo
zhang_xubo
Mijamind
quemingjian
laishenghao
dodders
zhoujingbnu
zhoujingbnu
刘伟
llzx373
douxin
ywzq1161327784
wangfeihuo
wangfeihuo
jinbo
l1azzzy
liuchangfeng
superblaker
libiao
libiao2024
jemappellehc
jemappellehc
siven_dev
siven_dev
CodeRejector
zankyfun4
未设置
最少人数
0
测试
吴岳川
wuyuechuan
chenxiaobin
chenxiaobin19
pengjiong
totaj
熊小军
xiong_xjun
wumingyuan
wumingyuan
陈栋
chendong76
zhaosiqi
dreamalloc
jeremytim
jeremytim
胡正超
gentle_hu
zhaojun
kelayzhao
huyinghao
yinghaohu
zhangxubo
zhang_xubo
Mijamind
quemingjian
laishenghao
dodders
zhoujingbnu
zhoujingbnu
刘伟
llzx373
douxin
ywzq1161327784
wangfeihuo
wangfeihuo
jinbo
l1azzzy
liuchangfeng
superblaker
libiao
libiao2024
jemappellehc
jemappellehc
siven_dev
siven_dev
CodeRejector
zankyfun4
未设置
最少人数
0
优先级
不指定
严重
主要
次要
不重要
标签
approved
opengauss-cla/yes
ci-pipeline-success
lgtm-totaj
lgtm-chenxiaobin19
sig/plugin
codecheck-success
关联 Issue
I92PY4
【测试类型:SQL功能】【测试版本:6.0.0】触发器body不支持select语句
I94H3U
【测试类型:SQL功能】【测试版本:6.0.0】show create trigger得到的SQL无法再次执行
Pull Request 合并后将关闭上述关联 Issue
里程碑
未关联里程碑
参与者
(5)
1
https://gitee.com/opengauss/Plugin.git
git@gitee.com:opengauss/Plugin.git
opengauss
Plugin
Plugin
点此查找更多帮助
搜索帮助
Git 命令在线学习
如何在 Gitee 导入 GitHub 仓库
Git 仓库基础操作
企业版和社区版功能对比
SSH 公钥设置
如何处理代码冲突
仓库体积过大,如何减小?
如何找回被删除的仓库数据
Gitee 产品配额说明
GitHub仓库快速导入Gitee及同步更新
什么是 Release(发行版)
将 PHP 项目自动发布到 packagist.org
仓库举报
回到顶部
登录提示
该操作需登录 Gitee 帐号,请先登录后再操作。
立即登录
没有帐号,去注册