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

Remove delayed_node #2014

Closed
1 of 17 tasks
abitmore opened this issue Sep 25, 2019 · 9 comments
Closed
1 of 17 tasks

Remove delayed_node #2014

abitmore opened this issue Sep 25, 2019 · 9 comments
Labels
5a Docs Needed Status specific to Documentation indicating the need for proper documentation to be added code cleanup

Comments

@abitmore
Copy link
Member

abitmore commented Sep 25, 2019

User Story
Features served by delayed_node now can be done with witness_node with appropriate configurations. The delayed_node program itself is somewhat buggy and lacking of features.

I suggest that we do this in next feature release.

Created https://github.com/bitshares/bitshares-core/wiki/Delayed-Node .

We may need to update other docs.

Impacts
Describe which portion(s) of BitShares Core may be impacted by your request. Please tick at least one box.

  • API (the application programming interface)
  • Build (the build process or something prior to compiled code)
  • CLI (the command line wallet)
  • Deployment (the deployment process after building such as Docker, Travis, etc.)
  • DEX (the Decentralized EXchange, market engine, etc.)
  • P2P (the peer-to-peer network for transaction/block propagation)
  • Performance (system or user efficiency, etc.)
  • Protocol (the blockchain logic, consensus, validation, etc.)
  • Security (the security of system or user data, etc.)
  • UX (the User Experience)
  • Other (please add below)

Additional Context (optional)
Add any other context about your request here.

CORE TEAM TASK LIST

  • Evaluate / Prioritize Feature Request
  • Refine User Stories / Requirements
  • Define Test Cases
  • Design / Develop Solution
  • Perform QA/Testing
  • Update Documentation
@abitmore abitmore added this to the 4.1.0 - Feature Release milestone Sep 25, 2019
@abitmore abitmore added 5a Docs Needed Status specific to Documentation indicating the need for proper documentation to be added code cleanup labels Sep 25, 2019
@abitmore
Copy link
Member Author

https://github.com/bitshares/bitshares-core/wiki/Delayed-Node

@ryanRfox
Copy link
Contributor

I support the removal of delayed_node as the plug in serves this functionality and we are able to remove technical debt from our project.

@xeroc please announce this on the mailing list as it will be a major change for existing node operators starting from 4.1.0 (target).

@MichelSantos Let's also add notification in the 4.0.0 Release Notes that this will be the final release containing the delayed_node application and reference the wiki for planning.

@abitmore
Copy link
Member Author

abitmore commented Sep 25, 2019

@ryanRfox IMHO 4.1.0 need to be renamed to 3.4.0 and be released before the consensus upgrade release. We need more time to finish 4.0.0, thus it's better to do a feature release even more releases before the consensus release. So it's not a good time to announce now for @xeroc. And we won't have the application in 4.0.0 release.

@ryanRfox
Copy link
Contributor

I am open to another Feature Release to the 3.0 Protocol as you suggest. Let's discuss on the planning call tomorrow. We can adjust the target release dates and content for the upcoming releases.

@oxarbitrage
Copy link
Member

Code to remove merged, new document linked in the programs readme. Keeping issue open for other doc changes needed until release.

@pmconrad
Copy link
Contributor

@cedar-book can you check the documentation websites for stuff about delayed_node, and adapt it accordingly? Thanks!

@cedar-book
Copy link

Hi, @pmconrad, I did not delete Delayed Node section and information. So, I added a section and a link. Please check bitshares/dev.bitshares.works#170

@pmconrad
Copy link
Contributor

Thanks. AFAICS the documentation was already using witness_node instead of delayed_node in most places, so the new hints aren't really needed.
Please close this issue after merging the remaining changes in bitshares/dev.bitshares.works#170.

@cedar-book
Copy link

I have updated and merged-->(bitshares/dev.bitshares.works#170)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5a Docs Needed Status specific to Documentation indicating the need for proper documentation to be added code cleanup
Projects
None yet
Development

No branches or pull requests

5 participants