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

[release-v0.2] maintenance backports #1338

Merged
merged 8 commits into from Dec 10, 2021

Conversation

chappjc
Copy link
Member

@chappjc chappjc commented Dec 8, 2021

Since both the v0.3 and v0.4 stable branches of dcrdex are targeting the unreleased Decred 1.7 (dcrd/dcrwallet), this cherry-picks a handful of minor bug fixes for the 0.2 release branch.

This also adds two new commits to facilitate this backporting:

  • dcr harness: allow specifying binary paths (e.g. DCRD=dcrd-1.6 DCRWALLET=dcrwallet-1.6 ./harness.sh)
  • bump versions to v0.2.3 in the applications and in the packaging script

chappjc and others added 8 commits December 7, 2021 16:44
A pseudo-upgrade was added prior to the first release and prior to
schema versioning that would create the matches.forgiven column
if it does not exist.  It has existed in the CREATE TABLE statement
since first release so this ALTER TABLE is always a no-op that just
results in a logged NOTICE:

[DBG] DB: pq: NOTICE (42701) - duplicate_column: column "forgiven" of relation "matches" already exists, skipping

The comments around this indicate the same, that this should have been
removed when actual upgrades were implemented. Schema v1 does
indeed include this forgiven column, and this change can thus remove
the ALTER TABLE statement that tries to re-add it on every startup.
"execute" any epoch orders in DB that may be left over from unclean
shutdown.  Whatever epoch they were in will not be seen again.
Copy link
Member

@JoeGruffins JoeGruffins left a comment

Choose a reason for hiding this comment

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

Also tested using 0.2.2 client with this server and updating with active trades. Looks good to me.

Copy link
Member

@itswisdomagain itswisdomagain left a comment

Choose a reason for hiding this comment

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

LGTM

@chappjc chappjc merged commit 66adad2 into decred:release-v0.2 Dec 10, 2021
@chappjc chappjc deleted the backports-0-2 branch December 10, 2021 00:55
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.

None yet

4 participants