登录
注册
开源
企业版
高校版
搜索
帮助中心
使用条款
关于我们
开源
企业版
高校版
私有云
模力方舟
登录
注册
代码拉取完成,页面将自动刷新
捐赠
捐赠前请先登录
取消
前往登录
扫描微信二维码支付
取消
支付完成
支付提示
将跳转至支付宝完成支付
确定
取消
Watch
不关注
关注所有动态
仅关注版本发行动态
关注但不提醒动态
31
Star
32
Fork
226
openGauss
/
Plugin
代码
Issues
50
Pull Requests
1
Wiki
统计
流水线
服务
JavaDoc
PHPDoc
质量分析
Jenkins for Gitee
腾讯云托管
腾讯云 Serverless
悬镜安全
阿里云 SAE
Codeblitz
SBOM
我知道了,不再自动展开
1020
支持ALTER TABLE ADD(col type), add(col type) 语法
已合并
pengjiong:test6
openGauss:master
pengjiong
创建于 2023-09-16 15:24
克隆/下载
HTTPS
SSH
复制
下载 Email Patch
下载 Diff 文件
<!-- 感谢您提交Pull Reqeust --> <!-- 提交说明: 请按照以下的模板提交PR,下列内容均为必填项。如果未补充对应内容,不允许提交PR。 --> 【标题】(请简要描述下实现的内容) 支持ALTER TABLE ADD(col type), add(Col type) 语法 【实现内容】: 支持ALTER TABLE ADD(col type), add(Col type) 语法 【根因分析】: 语法功能缺失,openGauss仅支持 ALTER TABLE ADD(col type, ...); 的用法,不支持和其他alter table命令再一起结合的 ALTER TABLE ADD(col type, ...), add(col type, ...), other_cmd; 【实现方案】: 参考MySQL的文档,观察MySQL支持的 alter命令。发现对于列定义带括号的场景,仅支持给出列名和列定义,不支持 first/alter,所以参考MySQL的语法,新增语法分支即可。  新增后发现和原始的 ALTER TABLE ADD(col type, ...); 产生语法冲突,通过分析可以新增的语法分支实际上已经包含了原始语法的功能了,所以可以把原始的语法删掉。另外由于单个的 alter_table_cmd 原本只能代表一个命令,比如加1列,加1个索引等,所以alter_table_cmd 定义为了Node类型。支持 ADD(col type, ...) 后, alter_table_cmd 能代表多个命令,比如加多列。更合适的做法可能是把 alter_table_cmd 改成 List 类型,但是考虑到修改后,所有 alter_table_cmd 的子命令返回值都要改,加个 list_make,改动很多,而且 alter_table_cmd 只有在 alter_table_cmds 分支有被使用,且List和Node 本质上可以通过nodetag区分,所以保持原有定义,在alter_table_cmd 和alter_table_cmds出加注释说明, alter_table_cmd 虽然定义为Node,但可能返回List,在 alter_table_cmds 处根据实际的nodetag,确定应该做list_make还是list_concat. 【关联需求或issue】: https://gitee.com/opengauss/Plugin/issues/I81VS8 【开发自验报告】: 1. 请附上自验结果(内容或者截图)  2. 是否可以添加fastcheck测试用例,如是,请补充fastcheck用例 是,已添加 3. 是否涉及资料修改,如是,在docs仓库补充资料 是,https://gitee.com/opengauss/docs/pulls/5688 4. 是否考虑支撑升级和在线扩容等扩展场景 不涉及 5. 是否考虑异常场景/并发场景/前向兼容/性能场景 新语法分支包含了被删掉的旧语法,不影响前向兼容,用例也已覆盖新旧两种语法 6. 是否对其他模块产生影响 不涉及 【其他说明】:
怎样手动合并此 Pull Request
git checkout master
git pull https://gitee.com/totaj/Plugin.git test6
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-wuyuechuan
lgtm-chenxiaobin19
sig/plugin
codecheck-success
关联 Issue
I81VS8
【测试类型:SQL功能】【测试版本:5.1.0】ALTER TABLE不支持ALTER TABLE t1 ADD COLUMN (f1 INT, f3 int), ADD COLUMN (f2 INT)语法
Pull Request 合并后将关闭上述关联 Issue
里程碑
未关联里程碑
参与者
(5)
Cherry-pick 提交
Cherry-pick 可以将
特定提交(Commit)
从某个分支挑选并应用到另一个分支,实现快速集成特定更改,而无需合并整个分支。
请选择应用 Cherry-pick 提交 (Commit) 的目标分支
新建分支
当前账号不存在 Fork 仓库,建议 cherry-pick 到 Fork 仓库。
Fork 仓库
提交列表
Commit SHA
Commit Message
基于 Cherry-pick 后的分支发起 Pull Request
取消
Cherry-pick
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 帐号,请先登录后再操作。
立即登录
没有帐号,去注册