Skip to content

Keep the most recent gobject votes only #2815

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

Merged
merged 4 commits into from
Apr 1, 2019

Conversation

UdjinM6
Copy link

@UdjinM6 UdjinM6 commented Mar 29, 2019

Also drops gobject getvotes rpc.

@UdjinM6 UdjinM6 added this to the 14.0 milestone Mar 29, 2019
nmarley
nmarley previously approved these changes Mar 29, 2019
Copy link

@nmarley nmarley left a comment

Choose a reason for hiding this comment

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

utACK

@UdjinM6 UdjinM6 force-pushed the keepthemostrecentvoteonly branch from b1f407f to e1cfe98 Compare March 29, 2019 19:23
@UdjinM6 UdjinM6 added the RPC Some notable changes to RPC params/behaviour/descriptions label Mar 29, 2019
codablock
codablock previously approved these changes Mar 30, 2019
Copy link

@codablock codablock left a comment

Choose a reason for hiding this comment

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

utACK, but not sure we should add this to v14

We don't need to store full vote data for old votes - we never relay them to other nodes anyway.
Still keep old hashes to avoid re-requesting data etc.
@UdjinM6 UdjinM6 force-pushed the keepthemostrecentvoteonly branch 2 times, most recently from 1a30eca to c16d8e0 Compare March 30, 2019 15:08
@UdjinM6 UdjinM6 force-pushed the keepthemostrecentvoteonly branch from c16d8e0 to a8e0d30 Compare March 30, 2019 15:37
@UdjinM6
Copy link
Author

UdjinM6 commented Mar 30, 2019

Fixed a bug (a8e0d30). Also removed timestamp fix from this PR and instead implemented a better version of it (hopefully) in #2819.

Copy link

@codablock codablock left a comment

Choose a reason for hiding this comment

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

re-utACK. And after thinking more about this, I'm fine with this getting into v14

@UdjinM6 UdjinM6 merged commit a87909e into dashpay:develop Apr 1, 2019
thephez added a commit to thephez/dash-docs that referenced this pull request Apr 8, 2019
thephez added a commit to dash-docs/dash-docs that referenced this pull request Apr 10, 2019
* RPC - Update masternode current and winner
 - Remove protocol, lastseen, activeseconds

* RPC - Remove gobject getvotes
 - Related to dashpay/dash#2815

* RPC - Protx diff update
 - Add merkleRootQuorums field

Related to dashpay/dash#2833

* RPC - listunspent update

Related to dashpay/dash#2727

* RPC - protx update_registrar update
 - Rename votingKeyAddr to votingAddress

Related to dashpay/dash#2747

* RPC - Add bls fromsecret

Related to dashpay/dash#2841

* RPC - Start adding quorum RPCs

* RPC - Add quorum info details Minor formatting updates

* RPC - Add placeholders for remaining quorum sub-commands

* RPC - Add quorum sign/hasrecsig/isconflicting details

* RPC - Add note to regtest only quorum RPCs

* RPC - Add more detail to quorum dkgstatus

* Typo fix

* Minor format and example updates

* RPC - Add more quorum dkgstatus details
 - Field descriptions

* RPC - Add quorum dkgstatus detail level 1 and 2 info

* RPC - protx update
 - Remove support for "0"

Related to dashpay/dash#2622
thephez added a commit to thephez/dash-docs that referenced this pull request May 17, 2019
* RPC - Update masternode current and winner
 - Remove protocol, lastseen, activeseconds

* RPC - Remove gobject getvotes
 - Related to dashpay/dash#2815

* RPC - Protx diff update
 - Add merkleRootQuorums field

Related to dashpay/dash#2833

* RPC - listunspent update

Related to dashpay/dash#2727

* RPC - protx update_registrar update
 - Rename votingKeyAddr to votingAddress

Related to dashpay/dash#2747

* RPC - Add bls fromsecret

Related to dashpay/dash#2841

* RPC - Start adding quorum RPCs

* RPC - Add quorum info details Minor formatting updates

* RPC - Add placeholders for remaining quorum sub-commands

* RPC - Add quorum sign/hasrecsig/isconflicting details

* RPC - Add note to regtest only quorum RPCs

* RPC - Add more detail to quorum dkgstatus

* Typo fix

* Minor format and example updates

* RPC - Add more quorum dkgstatus details
 - Field descriptions

* RPC - Add quorum dkgstatus detail level 1 and 2 info

* RPC - protx update
 - Remove support for "0"

Related to dashpay/dash#2622
thephez added a commit to dash-docs/dash-docs that referenced this pull request May 17, 2019
* RPC - Update masternode current and winner
 - Remove protocol, lastseen, activeseconds

* RPC - Remove gobject getvotes
 - Related to dashpay/dash#2815

* RPC - Protx diff update
 - Add merkleRootQuorums field

Related to dashpay/dash#2833

* RPC - listunspent update

Related to dashpay/dash#2727

* RPC - protx update_registrar update
 - Rename votingKeyAddr to votingAddress

Related to dashpay/dash#2747

* RPC - Add bls fromsecret

Related to dashpay/dash#2841

* RPC - Start adding quorum RPCs

* RPC - Add quorum info details Minor formatting updates

* RPC - Add placeholders for remaining quorum sub-commands

* RPC - Add quorum sign/hasrecsig/isconflicting details

* RPC - Add note to regtest only quorum RPCs

* RPC - Add more detail to quorum dkgstatus

* Typo fix

* Minor format and example updates

* RPC - Add more quorum dkgstatus details
 - Field descriptions

* RPC - Add quorum dkgstatus detail level 1 and 2 info

* RPC - protx update
 - Remove support for "0"

Related to dashpay/dash#2622
@UdjinM6 UdjinM6 deleted the keepthemostrecentvoteonly branch November 26, 2020 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RPC Some notable changes to RPC params/behaviour/descriptions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants