-
Notifications
You must be signed in to change notification settings - Fork 132
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
Add support for composite Id's #158
Comments
We depend on upstream feature availability in Spring Data Relational for composite Id's, see DATAJDBC-352. |
any idea when it will be done? |
No, we don’t have this feature scheduled yet as we’re busy with other tasks. A pre-requisite is the composite I’d support in Spring Data Relational as the R2DBC module depends on a shared library. |
Can you suggest any workaround for this until it's done? edit: I just skipped these annotations, used one element of the composite primary key as a "key" for Repository and used custom query with edit 2: I handled composite key with |
@pwittchen do you mind to explain how did you did it with Tuple2? |
Something like this: @Repository
public interface UserRepository extends R2dbcRepository<User, Tuple2<BigDecimal, String> {
//...
} I know, this is kind of workaround and won't work with Repository methods provided out of the box like |
this is still forgotten =/ |
@pwittchen Can you please give a reference to your code? |
@palaknagda Code related to my comment is proprietary and is not open-source. You can see a general idea in my code snippet in the earlier comment. Nevertheless, it does not solve this problem and query inside |
Any updates??? |
Still depends on spring-projects/spring-data-relational#574. A well-crafted pull request along with a decent design can always speed things up. |
More than 3 years later, no updates so far? |
No way to specify composite id for usage ReactiveCrudRepository
The text was updated successfully, but these errors were encountered: