Skip to content

Conversation

@zhangyongding
Copy link
Contributor

No description provided.

@coveralls
Copy link

coveralls commented Sep 18, 2025

Coverage Status

coverage: 96.822% (-0.4%) from 97.182%
when pulling 8992646 on zhangyongding:delete-redundant-code
into 044cf35 on huandu:master.

@huandu
Copy link
Owner

huandu commented Oct 10, 2025

这些代码不是冗余,是故意这么设计的。拿 CTE Builder 来说,里面的 query 是不能够在 build 的时候才 add 到 Args 里面去的,如果这么做,每次 build 都会带来副作用,即 Args 里面的变量会越来越多,这明显是不合适的。

@huandu huandu closed this Oct 10, 2025
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