Skip to content

Add support for backend compatibility (#483) - #611

Merged
austindrenski merged 1 commit into
npgsql:hotfix/2.1.2from
austindrenski:hotfix/2.1.2
Aug 19, 2018
Merged

Add support for backend compatibility (#483)#611
austindrenski merged 1 commit into
npgsql:hotfix/2.1.2from
austindrenski:hotfix/2.1.2

Conversation

@austindrenski

Copy link
Copy Markdown
Contributor

Backporting per #607.

@austindrenskiaustindrenski added this to the 2.1.2 milestone Aug 19, 2018
@austindrenskiaustindrenski self-assigned this Aug 19, 2018
@roji

roji commented Aug 19, 2018

Copy link
Copy Markdown
Member

Hmm, this definitely contains more than just the pure backwards compat feature - this is good example why it's important to separate refactoring and other changes into different commits. We ideally want to take only what is absolutely needed when backporting to a hotfix release.

But this looks safe enough I guess. OK for me.

@austindrenski

Copy link
Copy Markdown
ContributorAuthor

I considered only bringing forward the compatibility portion of this, but then thought it was worth bringing the fix forward too since this was introduced to address the translation breaking behavior for 9.x.

But noted on keeping these commits more limited in scope.

@austindrenski
austindrenski merged commit 2f4c05f into npgsql:hotfix/2.1.2Aug 19, 2018
@austindrenski
austindrenski deleted the hotfix/2.1.2 branch August 19, 2018 17:53
Sign up for freeto 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

@austindrenski@roji