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

Refactor UnresolvedName #17000

Merged
merged 6 commits into from
Jun 20, 2024
Merged

Conversation

ck89119
Copy link
Contributor

@ck89119 ck89119 commented Jun 19, 2024

What type of PR is this?

  • API-change
  • BUG
  • Improvement
  • Documentation
  • Feature
  • Test and CI
  • Code Refactoring

Which issue(s) this PR fixes:

issue #16300 https://github.com/matrixorigin/MO-Cloud/issues/3119

What this PR does / why we need it:

  • Add/Remove some New() functions
  • Remove Parts, whose information is included in CStrParts
  • Add get db/tbl/col name and nameOrigin functions
  • Add lower_case_table_names into BindContext

@matrix-meow matrix-meow added the size/XXL Denotes a PR that changes 2000+ lines label Jun 19, 2024
@mergify mergify bot requested a review from sukki37 June 19, 2024 08:37
@mergify mergify bot added the kind/refactor Code refactor label Jun 19, 2024
@mergify mergify bot merged commit 30d8ef2 into matrixorigin:1.2-dev Jun 20, 2024
16 of 18 checks passed
@ck89119 ck89119 deleted the refactor/UnresolvedName branch June 20, 2024 12:00
ck89119 added a commit to ck89119/matrixone that referenced this pull request Jun 20, 2024
- Add/Remove some New() functions
- Remove Parts, whose information is included in CStrParts
- Add get db/tbl/col name and nameOrigin functions
- Add lower_case_table_names into BindContext

Approved by: @daviszhen, @aunjgr, @iamlinjunhong, @aressu1985, @reusee, @heni02, @sukki37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/refactor Code refactor size/XXL Denotes a PR that changes 2000+ lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants