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

Drop table #456

Closed
wants to merge 2 commits into from
Closed

Drop table #456

wants to merge 2 commits into from

Conversation

bannuanzz
Copy link

What problem were solved in this pull request?

Issue Number: drop table

Problem:

What is changed and how it works?

由于词法解析和语法解析原本代码里就带有
我在此修改了stmt 和executor 然后再storage里修改了handle db 和table 具体的代码实现逻辑是table里面的内容
具体可以看注释
此pr还增加了一些方便调试的内容。

Other information

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Morri77 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@hnwyllmm
Copy link
Collaborator

hnwyllmm commented Oct 8, 2024

看起来是误提交。
不过.clang-tidy和clang-format的代码看起来挺有意思,可以提交上来吗?

@hnwyllmm hnwyllmm closed this Oct 8, 2024
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

Successfully merging this pull request may close these issues.

3 participants