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

merge from origin #1

Open
wants to merge 91 commits into
base: master
Choose a base branch
from
Open

merge from origin #1

wants to merge 91 commits into from

Conversation

wizizm
Copy link
Owner

@wizizm wizizm commented May 20, 2021

No description provided.

xormplus added 30 commits March 6, 2020 15:12
Use strings.Builder instead of builder.StringBuilder
Remove unuse get cols code
fix tests
* fix bug when insert multiple slices with customize table name

* fix tests on mssql

* fix tests
* fix arg conversion

* fix bugs

* fix bug on postgres

* use traditional positional parameters on insert into select

* remove unnecessary tests
* fix default value

* fix default value tags

* fix postgres default

* fix default on postgres

* fix default on postgres

* fix mssql default
* add tests for db metas

* add more tests

* fix bug on mssql
* fix sync2 with custom table name

* fix bug on postgres

* fix bug on postgres
* don't warn when bool column default is 1 but not true

* fix default case sensitive
xormplus and others added 30 commits July 31, 2020 18:36
change goracle  to godror
Fix mysql dialect error from invalid db identifier in orderby clause
refactor: rename the function name

feat: define xorm.FromDialectAndDB
[oracle]get comments from table/columns,check index length
[oracle]go-oci8 can handler time.Time, don't do anything
add support: omit condition when generate SQL condition from beans
fix generate Limit sql twice in genSelectSql() when call statement.Limit()
cacheget会先查询数据库中的pk,然后再查询cache,当并发数超过最大连接数时,查询pk的conn不会被释放,导致获取连接时锁死
bug fix: release conn after query pk in cacheget
bug fix #104 float32 to float64 error
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.

8 participants