Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Optimize parse constraint #4153

Merged
merged 4 commits into from
Mar 8, 2021
Merged

Optimize parse constraint #4153

merged 4 commits into from
Mar 8, 2021

Conversation

daheige
Copy link
Contributor

@daheige daheige commented Mar 7, 2021

gorm/schema/relationship.go ParseConstraint method optimization.
The following code is basically called in for. In order to avoid the performance problems caused by repeated compilation of regular expressions, it only needs to be done once outside, so optimization is done here.

@jinzhu jinzhu merged commit 02cb405 into go-gorm:master Mar 8, 2021
mittwillson pushed a commit to itering/gorm that referenced this pull request Sep 27, 2021
* for Config.cacheStore store PreparedStmtDB key

* invalid db error and value and invalid value length error (go-gorm#4151)

* support named params in Select API  (go-gorm#4142)

* adds support for named arguments in select

* changes clause identifies and adds test

* optimize match english letters and midline

Co-authored-by: Ratan Phayade <ratanphayade@users.noreply.github.com>
cgxxv pushed a commit to cgxxv/gorm that referenced this pull request Mar 25, 2022
* for Config.cacheStore store PreparedStmtDB key

* invalid db error and value and invalid value length error (go-gorm#4151)

* support named params in Select API  (go-gorm#4142)

* adds support for named arguments in select

* changes clause identifies and adds test

* optimize match english letters and midline

Co-authored-by: Ratan Phayade <ratanphayade@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants