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

Introduce replacement for ReactiveDataAccessStrategy #531

Open
mp911de opened this issue Jan 29, 2021 · 2 comments
Open

Introduce replacement for ReactiveDataAccessStrategy #531

mp911de opened this issue Jan 29, 2021 · 2 comments
Assignees
Labels
type: task A general task

Comments

@mp911de
Copy link
Member

mp911de commented Jan 29, 2021

No description provided.

@mp911de mp911de added the type: task A general task label Jan 29, 2021
@mp911de mp911de self-assigned this Jan 29, 2021
@mp911de
Copy link
Member Author

mp911de commented Feb 1, 2021

  • getAllColumns and getIdentifierColumns -> RelationalPersistentEntity
  • getOutboundRow -> UpdateMapper
  • getRowMapper -> direct usage via EntityRowMapper and R2dbcConverter

@pmaedel
Copy link

pmaedel commented Feb 6, 2021

I was able to use getAllColumns to create join assignments for a respective table alias:

inline fun <reified T : Any> ReactiveDataAccessStrategy.joinAssignments(tableAlias: String): String =
    getAllColumns(T::class.java).joinToString(", ") { column ->
        "$tableAlias.${column.reference.toLowerCase()} as ${tableAlias}_${column.reference.toLowerCase()}"
    }

Maybe it is a good idea to have this functionality integrated along the respective alias mapping requested in #448?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: task A general task
Projects
None yet
Development

No branches or pull requests

2 participants