Skip to content

Conversation

@yuyu1025
Copy link

I want to implement soft deletion step by step by modifying the methods of R2dbcEntityTemplate, while also achieving better scalability.

I plan to achieve this in several steps:

  1. Provide a query wrapper, allowing the implementation class for logical deletion to override this wrapper function.
  2. Split the doDelete method, separating the construction behavior from the final execution. The construction process can be overridden, and the implementation class for logical deletion is also provided to override it.

  - Introduce QueryWrapper interface for query modification
  - Add DefaultQueryWrapper as no-op implementation
  - Integrate query wrapper into R2dbcEntityTemplate operations
  - Support query customization for count, exists, select, update, delete operations

Signed-off-by: finger <[email protected]>
  - Introduce QueryWrapper interface for query modification
  - Add DefaultQueryWrapper as no-op implementation
  - Integrate query wrapper into R2dbcEntityTemplate operations
  - Support query customization for count, exists, select, update, delete operations

Signed-off-by: finger <[email protected]>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 12, 2025
@mp911de
Copy link
Member

mp911de commented Aug 14, 2025

Thanks for reaching out. We have't really intentions to introduce query post-processing at that level. Introducing an query augmentation mechanism requires a proper design scheme that also fits the JDBC part. We therefore do not want to continue with this PR.

@mp911de mp911de closed this Aug 14, 2025
@mp911de mp911de added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged labels Aug 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: declined A suggestion or change that we don't feel we should currently apply

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants