matrixorigin / matrixorigin/matrixone

[Tech Request]: Avoide using expr.String() or tree.String() during BuildPlan

Open
#22,335 0 comments 0 reactions 1 assignee Claimed by @aunjgr View on GitHub
kind/tech-request priority/p1
Dominant language
Go
Stars
1.9k
Forks
311
Avg merge
1d 3h
Merged PRs (30d)
768

Description

### Is there an existing issue for the same tech request?

- [x] I have checked the existing issues.

### Does this tech request not affect user experience?

- [x] This tech request doesn't affect user experience.

### What would you like to be added ?

```Markdown
Avode using expr.String() or tree.String() during BuildPlan.
Using ExprHash instead. Or optimized string() implementation if needed.
```

### Why is this needed ?

[pprof.mo-service.samples.cpu.129.pb.gz](https://github.com/user-attachments/files/21734141/pprof.mo-service.samples.cpu.129.pb.gz)

### Additional information

Image

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.