Skip to content

Updated spring-data-cosmos read me to fix javadoc errors and sb3 version#36065

Merged
kushagraThapar merged 1 commit into
Azure:feature/spring-boot-3from
kushagraThapar:update_spring_data_cosmos_sb_3_readme
Jul 25, 2023
Merged

Updated spring-data-cosmos read me to fix javadoc errors and sb3 version#36065
kushagraThapar merged 1 commit into
Azure:feature/spring-boot-3from
kushagraThapar:update_spring_data_cosmos_sb_3_readme

Conversation

@kushagraThapar

Copy link
Copy Markdown
Member

Updated spring-data-cosmos read me to fix javadoc errors and sb3 version

@azure-sdk

Copy link
Copy Markdown
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

azure-spring-data-cosmos

Mono<Boolean> existsByLastNameIsNull();

@Query(value = "SELECT * FROM a WHERE ARRAY_CONTAINS(@firstNames, a.firstName) ")
@Query(value = "SELECT * FROM a WHERE ARRAY_CONTAINS(@firstNames, a.firstName) order by a.id ")

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why changing the query? should we add another test for the order by?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is already another one for order by :)
Changing this one so that we can match the results 100% of the time always, currently they come unordered and can come in any order without any guarantee, causing the test to be flaky.

@xinlian12 xinlian12 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks

@kushagraThapar kushagraThapar merged commit 2816f36 into Azure:feature/spring-boot-3 Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

azure-spring All azure-spring related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants