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

network: Add capability filtered depth calculation #1787

Merged
merged 2 commits into from
Sep 20, 2019

Conversation

nolash
Copy link
Contributor

@nolash nolash commented Sep 20, 2019

This PR adds methods that enables depth calculations based on registered capability indices in the kademlia.

The score of this incremental change is to make it possible to do correct forwarding decisions for pss depending on whether the capability in pss enabling forwarding of behalf of the network is set or not.

@nolash nolash requested review from janos and zelig September 20, 2019 12:21
@nolash nolash requested a review from kortatu September 20, 2019 12:21
@nolash nolash moved this from Backlog to In review (includes Documentation) in Swarm Core - Sprint planning Sep 20, 2019
Copy link
Contributor

@kortatu kortatu left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@janos janos left a comment

Choose a reason for hiding this comment

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

I did not think that new depths would be so nice to add. It will be fun to add triggers per capability index, when that is needed.

func TestCapabilityNeighbourhoodDepth(t *testing.T) {
baseAddressBytes := RandomBzzAddr().OAddr
kad := NewKademlia(baseAddressBytes, NewKadParams())
cap_both := capability.NewCapability(42, 2)
Copy link
Member

Choose a reason for hiding this comment

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

No underscores in variable names.

@zelig zelig merged commit afd1d9f into ethersphere:master Sep 20, 2019
Swarm Core - Sprint planning automation moved this from In review (includes Documentation) to Done Sep 20, 2019
janos added a commit that referenced this pull request Sep 23, 2019
chadsr added a commit to chadsr/swarm that referenced this pull request Sep 23, 2019
* 'master' of github.com:ethersphere/swarm: (32 commits)
  network/stream: refactor cursors tests (ethersphere#1786)
  network: Add capabilities if peer from store does not have it (ethersphere#1791)
  Swap logger (ethersphere#1754)
  network: Add capability filtered depth calculation (ethersphere#1787)
  travis: remove go1.12 job (ethersphere#1784)
  cmd/swarm: correct bzznetworkid flag description (ethersphere#1761)
  network, pss: Capability in pss (ethersphere#1764)
  network/stream: handle nil peer in TestNodesExchangeCorrectBinIndexes (ethersphere#1779)
  protocols, retrieval: swap-enabled messages implement Price (ethersphere#1771)
  cmd/swarm-smoke: fix waitToPushSynced connection closing (ethersphere#1781)
  cmd/swarm: simplify testCluster.StartNewNodes (ethersphere#1777)
  build: increase golangci-lint deadline (ethersphere#1778)
  docker: ignore build/bin when copying files (ethersphere#1780)
  swap: fix and rename Peer.getLastSentCumulativePayout (ethersphere#1769)
  network/stream: more resilient TestNodesCorrectBinsDynamic (ethersphere#1776)
  network: Add Capabilities to Kademlia database (ethersphere#1713)
  network: add own address to KademliaInfo (ethersphere#1775)
  pss: Refactor. Step 2. Refactor forward cache (ethersphere#1742)
  all: configurable payment/disconnect thresholds (ethersphere#1729)
  network/stream/v2: more resilient TestNodesExchangeCorrectBinIndexes (ethersphere#1760)
  ...
@skylenet skylenet added this to the 0.5.0 milestone Sep 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

5 participants