matrixorigin / matrixorigin/matrixone

[Refactoring]: rename pkg/sql/plan to planner

Open
#23,589 0 comments 0 reactions 1 assignee Claimed by @aunjgr View on GitHub
needs-triage
Dominant language
Go
Stars
1.9k
Forks
311
Avg merge
1d 3h
Merged PRs (30d)
768

Description

### Why do you want to refactor this code?

```Markdown
There are two packages named "plan", pkg/pb/plan and pkg/sql/plan

It causes confusion and inconsistency throughout the codebase
```

### Describe the solution you'd like

Move pkg/sql/plan to pkg/sql/planner and rename the package.

Keep pkg/pb/plan as is.

### Describe alternatives you've considered

_No response_

### Additional information

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.