Skip to content

[PARKED] RIP dir lookup#131

Open
vyzo wants to merge 1 commit into
masterfrom
vyzo-no-dir-lookup
Open

[PARKED] RIP dir lookup#131
vyzo wants to merge 1 commit into
masterfrom
vyzo-no-dir-lookup

Conversation

@vyzo
Copy link
Copy Markdown
Contributor

@vyzo vyzo commented Feb 10, 2017

As planned when we originally introduced DHT lookups, we can now retire directory lookups in mcnode.

@vyzo vyzo added this to the v1.6 milestone Feb 10, 2017
@parkan
Copy link
Copy Markdown
Contributor

parkan commented Feb 10, 2017

Hmm I'm not sure we've definitively decided to do this, I can still see directories being useful for certain scenarios.

@vyzo
Copy link
Copy Markdown
Contributor Author

vyzo commented Feb 10, 2017

Oh, we are not ditching directories altogether, and we still publish peer info when we register for non-dht clients to discover us. This just prunes the directory lookup in concat in favor of using the DHT.

@parkan
Copy link
Copy Markdown
Contributor

parkan commented Feb 10, 2017

let's sleep on this one

@vyzo
Copy link
Copy Markdown
Contributor Author

vyzo commented Feb 13, 2017

There are some arguments for keeping the dir lookup, at least for a while longer:

  • internal lookups for clusters
  • aleph nodes can go public without integrating with the dht

So let's keep it parked and revisit later on.

@vyzo vyzo removed this from the v1.6 milestone Feb 13, 2017
@parkan parkan changed the title RIP dir lookup [PARKED] RIP dir lookup Feb 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants