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

remove unused code #4051

Merged
merged 1 commit into from
Mar 11, 2022
Merged

remove unused code #4051

merged 1 commit into from
Mar 11, 2022

Conversation

mklzl
Copy link
Contributor

@mklzl mklzl commented Mar 11, 2022

Removed an unused exception : AnalysisException

What type of PR is this:

  • bug
  • feature
  • enhancement
  • others

Which issues of this PR fixes :

Fixes #1742

Problem Summary(Required) :

some unused code can be deleted

Removed an unused exception : AnalysisException
@wanpengfei-git
Copy link
Collaborator

[FE PR Coverage check]

😍 pass : 0 / 0 (0%)

@kangkaisen kangkaisen added the Approved Ready to merge label Mar 11, 2022
@dirtysalt dirtysalt merged commit 76a7f06 into StarRocks:main Mar 11, 2022
jaogoy pushed a commit to jaogoy/starrocks that referenced this pull request Nov 15, 2023
调整文中四处"Colocation Join"为"Colocate Join"

(cherry picked from commit e824594)

Co-authored-by: 吴梦龙 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved Ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Good First Issues
4 participants