Skip to content

Fixed issue when IN query statement didn't work with Long values (cherry pick from main)#36267

Merged
rinatmini merged 3 commits into
feature/spring-boot-3from
users/rinatmini/spring3cherrypick
Aug 7, 2023
Merged

Fixed issue when IN query statement didn't work with Long values (cherry pick from main)#36267
rinatmini merged 3 commits into
feature/spring-boot-3from
users/rinatmini/spring3cherrypick

Conversation

@rinatmini

Copy link
Copy Markdown
Contributor

Description

Related issue: #36209

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

* Fixed when IN query statement didn't work with Long values

* Warn fix

* Build warnings were fixed

* Added more tests + resolved PR comments
@github-actions github-actions Bot added the azure-spring All azure-spring related issues label Aug 7, 2023
@rinatmini rinatmini changed the title Fixed issue when IN query statement didn't work with Long values (cherry pick for main) Fixed issue when IN query statement didn't work with Long values (cherry pick from main) Aug 7, 2023
@rinatmini

Copy link
Copy Markdown
Contributor Author

run /azp java - spring - cosmos

1 similar comment
@rinatmini

Copy link
Copy Markdown
Contributor Author

run /azp java - spring - cosmos

@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

@rinatmini

Copy link
Copy Markdown
Contributor Author

/azp run java - spring - cosmos

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@kushagraThapar kushagraThapar 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 @rinatmini

Comment thread sdk/spring/azure-spring-data-cosmos/CHANGELOG.md Outdated
@rinatmini
rinatmini merged commit c497e4f into feature/spring-boot-3 Aug 7, 2023
@rinatmini
rinatmini deleted the users/rinatmini/spring3cherrypick branch August 7, 2023 18:17
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