Skip to content

Reuse Dialect support provided by Spring Data Relational #125

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

Closed
wants to merge 4 commits into from

Conversation

mp911de
Copy link
Member

@mp911de mp911de commented May 22, 2019

We now reuse the existing Dialect infrastructure provided by Spring Data Relational to enhance it for R2DBC specifics such as bind markers. This allows us to provide only the R2DBC-specific extensions through R2dbcDialect. We entirely rely on the provided rendering infrastructure and can remove our own tweaks.


Related ticket: #55.

mp911de added 4 commits May 22, 2019 09:42
We now reuse the existing Dialect infrastructure provided by Spring Data Relational to enhance it for R2DBC specifics such as bind markers.
@mp911de mp911de added this to the 1.0 M3 milestone May 22, 2019
@mp911de mp911de requested a review from schauder May 22, 2019 08:23
schauder pushed a commit that referenced this pull request May 22, 2019
We now reuse the existing Dialect infrastructure provided by Spring Data Relational to enhance it for R2DBC specifics such as bind markers.

Original pull request: #125.
schauder pushed a commit that referenced this pull request May 22, 2019
Original pull request: #125.
schauder pushed a commit that referenced this pull request May 22, 2019
Original pull request: #125.
schauder added a commit that referenced this pull request May 22, 2019
Minor changes to documentation in order to avoid <literal code>``s constructs as recommended by our style guide.

Original pull request: #125.
@schauder
Copy link
Contributor

That's merged.

@schauder schauder closed this May 22, 2019
@mp911de mp911de deleted the issue/gh-55 branch September 5, 2019 10:16
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.

2 participants