feat: support limit in spring data repository

[shouwn]

Motivation

• Spring Data supports a Limit class as a parameter in Repository since version 3.2.
• However, the Kotlin JDSL takes Spring Boot 2.7.9 and 3.0.3 as the minimum compatible version, which makes it difficult to support the Limit class.

Modifications

• Add the parameters offset: Int, limit: Int to make Limit available in the Repository even for the minimum compatible Spring Boot version supported by the Kotlin JDSL.

Result

• Users will be able to use offset and limit when using the findAll method of the KotlinJdslJpqlExecutor interface.

bookRepository.findAll(limit = 1) {
    select(
        path(Book::isbn),
    ).from(
        entity(Book::class),
        join(entity(BookPublisher::class)).on(path(BookPublisher::book).eq(entity(Book::class))),
    ).where(
        path(BookPublisher::publisherId).eq(3),
    )
}

bookRepository.findAll(offset = 1, limit = 1) {
    select(
        path(Book::isbn),
    ).from(
        entity(Book::class),
        join(entity(BookPublisher::class)).on(path(BookPublisher::book).eq(entity(Book::class))),
    ).where(
        path(BookPublisher::publisherId).eq(3),
    )
}

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 75.00000% with 8 lines in your changes missing coverage. Please review.

Project coverage is 91.73%. Comparing base (025f6c1) to head (e65f28c).

Files with missing lines	Patch %	Lines
...ata/jpa/javax/repository/KotlinJdslJpqlExecutor.kt	33.33%	4 Missing ⚠️
...ring/data/jpa/repository/KotlinJdslJpqlExecutor.kt	33.33%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #764      +/-   ##
==========================================
- Coverage   91.96%   91.73%   -0.24%     
==========================================
  Files         335      337       +2     
  Lines        3449     3447       -2     
  Branches      205      209       +4     
==========================================
- Hits         3172     3162      -10     
- Misses        215      223       +8     
  Partials       62       62              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[cj848]

Both offset and limit are Integer types, so how about specifying the parameter names? I think it would be better to include them when calling the method, just like in the example you wrote. The rest of the code is the same.

--- korean
offset, limit 모두 Integer 타입인데 파라메터의 이름을 명시하는게 어떤가요? 예제로 적어놓으신 부분과 동일하게 메소드 호출시에도 넣어두는게 좋을 것 같습니다. 나머지 코드도 동일합니다.