Hari Ohm Prasath
Hari Ohm Prasath
Hi @kiview, We have a similar requirement and based on my understanding we need to fix this in two places: ## Changes to cache URL We need to make sure...
Hi @sneakythr0ws, Looks like you might be missing this dependency in your gradle file which could be causing this issue: `testImplementation "org.testcontainers:r2dbc:1.17.3"` Can you add this and give it a...
> There are quite a few unused fields that I am not sure if just should be deleted or some parts of implementation is still missing. > > Missing integration...
> Reference documentation added, package-info with nullability annotation added, no infrastructure required if cloudmap namespace or service doesnt exist the cloudmap connector module will automatically create them
> > There are quite a few unused fields that I am not sure if just should be deleted or some parts of implementation is still missing. > > Missing...
> @hariohmprasath thanks for an update. We still need to have an integration test that checks everything all together. Something like `ParameterStoreConfigDataLoaderIntegrationTests` - does not need to use `SpringApplication`, can...
Hi @maciejwalkowiak, I have added integration tests to test service discovery and registration for both the platforms (`ECS` and `EKS`). * `EcsCloudMapIntegrationTest.java` - Integration tests for `ECS` platform * `EksCloudMapIntegrationTest.java`...
Hi @maciejwalkowiak, Just checking to see whether you had a chance to review the changes, let me know if you have any questions/suggestions. Thanks
@koenpunt, Interesting, I just ran the test case under `org.springframework.data.r2dbc.repository.query.PartTreeR2dbcQueryUnitTests#createQueryWithPessimisticWriteLock()` in [PartTreeR2dbcQueryUnitTests.java.java](https://github.com/spring-projects/spring-data-relational/blob/b46e34263eed96462cf3c0db93cf7ab2132610a5/spring-data-r2dbc/src/test/java/org/springframework/data/r2dbc/repository/query/PartTreeR2dbcQueryUnitTests.java) which matches with your usecase and the code seems to be generating the final query with the `FOR...
Hi @loolzaaa, Your suggestion makes sense for me, since its a simple change I have submitted a PR. But I will wait to hear from maintainers of this repo before...