Skip to content
This repository has been archived by the owner on Aug 23, 2020. It is now read-only.

Use neighbors according to heartbeat #1798

Open
kwek20 opened this issue Mar 16, 2020 · 1 comment · May be fixed by #1830
Open

Use neighbors according to heartbeat #1798

kwek20 opened this issue Mar 16, 2020 · 1 comment · May be fixed by #1830
Milestone

Comments

@kwek20
Copy link
Contributor

kwek20 commented Mar 16, 2020

Description

With #1792 IRI will support heartbeat checks for neighbors. We only want to connect to neighbors who (partially) support our milestone range

Motivation

Requesting transactions from neighbors who are known not to have the milestone the tx is in, takes a needless toll on network usage

Requirements

  • Modify Neighbor setup to not request transactions we know the neighbor doesn't have
  • Keep support for neighbors requesting transactions (Thus not dropping them if we aren't interested)

Open Questions (optional)

  • How often do we check this?
@iotaledger iotaledger deleted a comment from tylerw1369 Mar 16, 2020
@GalRogozinski GalRogozinski added C-Network L-Groom This issue needs to be groomed labels Mar 26, 2020
@GalRogozinski
Copy link
Contributor

Broadcast it every minute or check what hornet does

@GalRogozinski GalRogozinski removed the L-Groom This issue needs to be groomed label Mar 26, 2020
@GalRogozinski GalRogozinski added this to the Mardi Gras milestone Mar 30, 2020
@GalRogozinski GalRogozinski changed the title Use neighbors according to hearthbeat Use neighbors according to heartbeat Mar 31, 2020
@acha-bill acha-bill linked a pull request Apr 8, 2020 that will close this issue
6 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants