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

Custom query with page will generate duplicate order cypher #2940

Closed
abccbaandy opened this issue Aug 20, 2024 · 1 comment
Closed

Custom query with page will generate duplicate order cypher #2940

abccbaandy opened this issue Aug 20, 2024 · 1 comment
Assignees
Labels
type: bug A general bug

Comments

@abccbaandy
Copy link

abccbaandy commented Aug 20, 2024

//page config
PageRequest pageable = PageRequest.of(0, 10, Sort.by(Sort.Order.asc("code")));

//Neo4jRepository
Page<Node> findByCode(String code, Pageable page);

will generate something like

ORDER BY node.code, node.code LIMIT 10

But with build-in function

.findAll(pageable);

It works fine.

test with spring boot 3.3.2

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 20, 2024
@michael-simons
Copy link
Collaborator

It should not do any harm, but it's ugly. I'll have a look.

michael-simons added a commit that referenced this issue Aug 20, 2024
If it does or is equal to, don't add it a second time.

Fixes #2940
michael-simons added a commit that referenced this issue Aug 20, 2024
If it does or is equal to, don't add it a second time.

Fixes #2940
@michael-simons michael-simons added the type: bug A general bug label Aug 20, 2024
@michael-simons michael-simons self-assigned this Aug 20, 2024
@michael-simons michael-simons removed the status: waiting-for-triage An issue we've not yet triaged label Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

No branches or pull requests

3 participants