Update FindAndCount #2467

Open

heliosgo wants to merge 1 commits from heliosgo/xorm:main into main

pull from: heliosgo/xorm:main

merge into: xorm:main

xorm:v1

xorm:main

xorm:lunny/vul_scan

xorm:lunny/oracle_ci

xorm:lunny/fix_distinct_count

xorm:lunny/postgres_for_update

xorm:lunny/executors

xorm:lunny/dm_ci

xorm:lunny/dm_upgrade

Conversation 2 Commits 1 Files Changed 1 +11

heliosgo commented 2024-07-11 14:00:50 +00:00

First-time contributor

Copy Link

某些情况下 FindAndCount 可以减少一次 Count 调用

某些情况下 FindAndCount 可以减少一次 Count 调用

heliosgo added 1 commit 2024-07-11 14:00:51 +00:00

Update FindAndCount 0a2fdb6137

lunny commented 2024-07-17 03:06:38 +00:00

Owner

Copy Link

I don't think it's right especially since you have some query conditions.

I don't think it's right especially since you have some query conditions.

heliosgo force-pushed main from 0a2fdb6137 to c6b18fbe5d 2024-07-17 06:51:50 +00:00 Compare

heliosgo force-pushed main from c6b18fbe5d to a78276bce8 2024-07-17 07:02:03 +00:00 Compare

heliosgo commented 2024-07-17 07:09:56 +00:00

Author

First-time contributor

Copy Link

I don't think it's right especially since you have some query conditions.

不好意思，有两处边界确实存在问题，已修复。对于查询条件的影响不是很理解，有例子吗？

> I don't think it's right especially since you have some query conditions. 不好意思，有两处边界确实存在问题，已修复。对于查询条件的影响不是很理解，有例子吗？

Some checks failed

Hide all checks

test cockroach / test cockroach (pull_request) Has been cancelled

Required

Details

test mariadb / test mariadb (pull_request) Has been cancelled

Required

Details

test mssql / test mssql with collation (pull_request) Has been cancelled

Required

Details

test mssql / test mssql (pull_request) Has been cancelled

Required

Details

test mysql / test mysql (pull_request) Has been cancelled

Required

Details

test mysql8 / test mysql8 (pull_request) Has been cancelled

Required

Details

test postgres / test postgres (pull_request) Has been cancelled

Required

Details

test sqlite / unit test & test sqlite (pull_request) Has been cancelled

Required

Details

test tidb / test tidb (pull_request) Has been cancelled

Required

Details

Some required checks were not successful.

You are not authorized to merge this pull request.

View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u main:heliosgo-main

git checkout heliosgo-main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

backport/done

  

backport/v1

  

blocked

  

db

oracle

  

db

sqlserver

  

duplicate

  

feature

cache

  

frontport/done

  

frontport/main

  

invalid

  

kind

breaking

  

kind

bug

  

kind

build

  

kind

dependencies

  

kind

docs

  

kind

driver

  

kind

enhancement

  

kind

feature

  

kind

performance

  

kind

proposal

  

kind

question

  

kind

refactor

  

kind

testing

  

need

feedback

  

need

test

  

proposal:accepted

  

RaspBerry Pi

  

regression

  

skip-changelog

  

upstream

  

wip

  

wontfix

No Label

backport/done

backport/v1

blocked

db

oracle

db

sqlserver

duplicate

feature

cache

frontport/done

frontport/main

invalid

kind

breaking

kind

bug

kind

build

kind

dependencies

kind

docs

kind

driver

kind

enhancement

kind

feature

kind

performance

kind

proposal

kind

question

kind

refactor

kind

testing

need

feedback

need

test

proposal:accepted

RaspBerry Pi

regression

skip-changelog

upstream

wip

wontfix

Milestone

Clear milestone

No items

No Milestone

Assignees

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: xorm/xorm#2467

No description provided.