This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Rewrite peers module - Closes #409 #452

Merged

karmacoma merged 40 commits into development from 409_rewrite-peers-module

Mar 2, 2017

Contributor

karmacoma commented Feb 27, 2017 •

edited

Loading

Phase 1 of 2: Move peers management to memory instead of database. Closes #409.

Original pull request: #441

4miners and others added 30 commits

February 5, 2017 05:58


          Retrieve git commit info only once on app start

901a28d


          Improved documentation

b2efb03


          Fix unhandled promise rejection

4c6b835


          Move peers management to memory instead of database


          Save peers to db on node shutdown and load them on node start

c1373db


          Move peers management to memory instead of database

2f028a5


          Save peers to db on node shutdown and load them on node start

46c86c9


          Merge branch '409_rewrite-peers-module' of https://github.com/4miners…

dd4725b

…/lisk into 409_rewrite-peers-module


          Start sync when peers ready, improve logging

d2a98e6


          Move peers management to memory instead of database

41d1ca1


          Save peers to db on node shutdown and load them on node start

31edbb4


          Merge remote-tracking branch 'origin/409_rewrite-peers-module' into 4…

22c6baa

…09_rewrite-peers-module


          unit tests of peers module

854aeca


          add extra peers unit tests, resolve transaction collision tests deter…

ccd36c9

…minism


          load all modules for unit tests, sync peers loader unit tests

fcf2a60


          load all modules for unit tests, sync peers loader unit tests

1ec011b


          Improved peers logic

277bf6e


          Merge with development

f41df57


          Merge branch 'development' of https://github.com/LiskHQ/lisk into 409…

ff68191

…_rewrite-peers-module


          Merge branch '409_rewrite-peers-module' of https://github.com/4miners…

27f79cb

…/lisk into 409_rewrite-peers-module


          peers modules tests rewrite

f9cca3b


          peer logic unit tests

64d69fb


          unit tests for logic/peers

a0def7a


          unit tests for logic/peers

f80b540


          Merge remote-tracking branch 'origin/409_rewrite-peers-module' into 4…

04eb2db

…09_rewrite-peers-module


          ban unban remove tests for peers logic

0bb62c4


          ban unban remove tests for peers logic

a3c6175


          add extra tests for peer and peers logic

a8142ba


          Merge branch '4miners/409_rewrite-peers-module'

2e825c7


          Removing --coverage option

9e20493

karmacoma requested a review from MaciejBaj

February 27, 2017 12:45

karmacoma self-assigned this

coveralls commented Feb 27, 2017 •

edited

Loading

Coverage increased (+0.9%) to 65.745% when pulling ffe05b0 on 409_rewrite-peers-module into dd51e5a on development.

MaciejBaj reviewed

View reviewed changes

test/common/modules.js Outdated

		@@ -0,0 +1,73 @@
		var sinon = require('sinon');

Contributor

MaciejBaj Feb 28, 2017

file should be deleted as it is not used anywhere (it was added but then)


          remove node from unit tests, remove unused modules stubs file

3e8eb41

MaciejBaj approved these changes

View reviewed changes

coveralls commented Feb 28, 2017 •

edited

Loading

Coverage increased (+0.9%) to 65.745% when pulling 3e8eb41 on 409_rewrite-peers-module into dd51e5a on development.

Oliver Beddows added 2 commits

February 28, 2017 14:11


          Returning setImmediate

56759e9


          Removing empty lines

7f92df6

coveralls commented Feb 28, 2017 •

edited

Loading

Coverage increased (+0.5%) to 65.369% when pulling 7f92df6 on 409_rewrite-peers-module into dd51e5a on development.

10 similar comments

coveralls commented Feb 28, 2017

Coverage increased (+0.5%) to 65.369% when pulling 7f92df6 on 409_rewrite-peers-module into dd51e5a on development.

coveralls commented Feb 28, 2017 •

edited

Loading

Coverage increased (+0.5%) to 65.369% when pulling 7f92df6 on 409_rewrite-peers-module into dd51e5a on development.

coveralls commented Feb 28, 2017

Coverage increased (+0.5%) to 65.369% when pulling 7f92df6 on 409_rewrite-peers-module into dd51e5a on development.

coveralls commented Feb 28, 2017

Coverage increased (+0.5%) to 65.369% when pulling 7f92df6 on 409_rewrite-peers-module into dd51e5a on development.

coveralls commented Feb 28, 2017

Coverage increased (+0.5%) to 65.369% when pulling 7f92df6 on 409_rewrite-peers-module into dd51e5a on development.

coveralls commented Feb 28, 2017

Coverage increased (+0.5%) to 65.369% when pulling 7f92df6 on 409_rewrite-peers-module into dd51e5a on development.

coveralls commented Feb 28, 2017

Coverage increased (+0.5%) to 65.369% when pulling 7f92df6 on 409_rewrite-peers-module into dd51e5a on development.

coveralls commented Feb 28, 2017

Coverage increased (+0.5%) to 65.369% when pulling 7f92df6 on 409_rewrite-peers-module into dd51e5a on development.

coveralls commented Feb 28, 2017

Coverage increased (+0.5%) to 65.369% when pulling 7f92df6 on 409_rewrite-peers-module into dd51e5a on development.

coveralls commented Feb 28, 2017 •

edited

Loading

Coverage increased (+0.5%) to 65.369% when pulling 7f92df6 on 409_rewrite-peers-module into dd51e5a on development.

karmacoma changed the title ~~Rewrite peers module - #409~~ Rewrite peers module - Closes #409

karmacoma merged commit 07153fc into development

karmacoma added ready and removed type: bug chore type: docs type: improvement type: test labels

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet