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

支持informix,以便可以适配gbase #140

Merged
merged 2 commits into from
Mar 30, 2024
Merged

Conversation

zhangyongding
Copy link
Contributor

No description provided.

@coveralls
Copy link

Coverage Status

coverage: 96.304% (-0.03%) from 96.336%
when pulling cd34220 on zhangyongding:master
into a5eff69 on huandu:master.

@huandu
Copy link
Owner

huandu commented Mar 30, 2024

非常感谢你的 PR。

对于 Informix 我一点了解都没有,我问了一下 Kimi,看起来有些语法差异,主要是对于 LIMIT OFFSET 的处理,Informix 不支持这种写法,不过我从代码中似乎没看到,看看怎么能完善一下吧。

这是我咨询 Kimi 的结果,不确定对错,欢迎反馈。Kimi:Informix 和 MySQL 的语法差异

@zhangyongding
Copy link
Contributor Author

zhangyongding commented Mar 30, 2024 via email

@huandu
Copy link
Owner

huandu commented Mar 30, 2024

赞,我刚刚没注意到,确实是已经加上了。

@huandu huandu removed the question label Mar 30, 2024
@huandu huandu merged commit c35040f into huandu:master Mar 30, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants