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

Reactive MS SQL client support #249

Merged
merged 9 commits into from
Sep 13, 2021
Merged

Conversation

fedinskiy
Copy link
Contributor

@fedinskiy fedinskiy commented Sep 9, 2021

Reactive MS SQL client support as tech preview. Part of QUARKUS-1081

Commits will be squashed before or during the merge

@fedinskiy fedinskiy marked this pull request as draft September 9, 2021 08:41
@fedinskiy fedinskiy linked an issue Sep 9, 2021 that may be closed by this pull request
@fedinskiy fedinskiy marked this pull request as ready for review September 10, 2021 10:57
Copy link
Contributor

@Sgitario Sgitario left a comment

Choose a reason for hiding this comment

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

Also, rebase your PR.

Copy link
Contributor

@Sgitario Sgitario left a comment

Choose a reason for hiding this comment

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

Just one minor comment to add ++ please rebase your BR, at the moment, it cannot be merged because there are conflicts.

@Sgitario
Copy link
Contributor

run tests

@Sgitario
Copy link
Contributor

OCP are not working due to:

  • We're running out of space
  • The Jenkins nodes is not configured to use the RH registry
    Merging

@Sgitario Sgitario merged commit c208728 into quarkus-qe:main Sep 13, 2021
@Sgitario
Copy link
Contributor

@fedinskiy please backport it to 2.2

@Sgitario Sgitario mentioned this pull request Sep 13, 2021
fedinskiy added a commit to fedinskiy/quarkus-test-suite that referenced this pull request Sep 13, 2021
* +Removed unneeded parameters

* ~Test moved to integration tests

* +Coverage for MS SQL; support for not using database in SQL requests

* +Support of new API

* +Full coverage of MS SQL

* ~Refactoring

* +Temporary fix

* ~Format
fedinskiy added a commit to fedinskiy/quarkus-test-suite that referenced this pull request Sep 13, 2021
* +Removed unneeded parameters

* ~Test moved to integration tests

* +Coverage for MS SQL; support for not using database in SQL requests

* +Support of new API

* +Full coverage of MS SQL

* ~Refactoring

* +Temporary fix

* ~Format
Sgitario pushed a commit that referenced this pull request Sep 13, 2021
* +Removed unneeded parameters

* ~Test moved to integration tests

* +Coverage for MS SQL; support for not using database in SQL requests

* +Support of new API

* +Full coverage of MS SQL

* ~Refactoring

* +Temporary fix

* ~Format
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Coverage for reactive MS SQL client
2 participants