Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

不支持mysql8.0--json添加索引的语法 #600

Open
zdm-wyx opened this issue Aug 23, 2023 · 5 comments
Open

不支持mysql8.0--json添加索引的语法 #600

zdm-wyx opened this issue Aug 23, 2023 · 5 comments

Comments

@zdm-wyx
Copy link

zdm-wyx commented Aug 23, 2023

描述
对问题简单清晰的描述
使用goInception-v1.3.0-72,检查mysql8.0-json添加索引时,返回语法错误,但是在mysql8.0中执行时没有问题
重现
模拟重现的步骤
image

image

环境

  • 数据库: [mysql]
  • 版本: [例如 8.0]
  • goInception-v1.3.0-72

参数
可能与问题相关的设置参数

@zdm-wyx
Copy link
Author

zdm-wyx commented Aug 24, 2023

@hanchuanchuan

@qiqizjl
Copy link

qiqizjl commented Aug 26, 2023

一样的问题 看起来好像是解析器部分没有支持 json

@zdm-wyx
Copy link
Author

zdm-wyx commented Aug 28, 2023

我看它那个语句直接去内嵌的tidb执行了,好像tidb不支持这个语法,只能等作者升级go inception了?

@qiqizjl
Copy link

qiqizjl commented Aug 28, 2023

我看它那个语句直接去内嵌的tidb执行了,好像tidb不支持这个语法,只能等作者升级go inception了?

对~ 我们目前的需求暂时还是 default 支持 json_array。暂时内部 fork 了一个版本魔改了https://github.com/hanchuanchuan/goInception/blob/master/parser/parser.y 看起来他的语句都是使用 yacc解析的。主要是 tidb 对 json 的支持没有 MySQL 的全面。所以看起来只能自己改了。。

@zdm-wyx
Copy link
Author

zdm-wyx commented Aug 28, 2023

我看它那个语句直接去内嵌的tidb执行了,好像tidb不支持这个语法,只能等作者升级go inception了?

对~ 我们目前的需求暂时还是 default 支持 json_array。暂时内部 fork 了一个版本魔改了https://github.com/hanchuanchuan/goInception/blob/master/parser/parser.y 看起来他的语句都是使用 yacc解析的。主要是 tidb 对 json 的支持没有 MySQL 的全面。所以看起来只能自己改了。。

谢谢🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants