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

ipns resolve local name very slow #1831

Closed
krl opened this issue Oct 12, 2015 · 7 comments
Closed

ipns resolve local name very slow #1831

krl opened this issue Oct 12, 2015 · 7 comments
Assignees
Labels
kind/bug A bug in existing code (including security flaws) topic/ipns Topic ipns

Comments

@krl
Copy link
Contributor

krl commented Oct 12, 2015

$ time ipfs name resolve
/ipfs/QmXQ5czKP2mhQrF2RtfiVv9sQ97MpWiyhSTTXgefgQdmHU
real    1m0.097s

@whyrusleeping
This should probably read from the local known value of the ipns key, no?

It was pretty much instant before.

@whyrusleeping
Copy link
Member

yeah, this behaviour was changed to query the network for records and select the best one. I agree that resolving the local name should be quick, but i'm not sure on the UX for this.

@jbenet
Copy link
Member

jbenet commented Oct 12, 2015

wow, is it really taking 1min to go through the whole dht? exhausting the queriable set should exit this quicker.

maybe the queries are taking a long time as setting up new connections is very slow.

we should really get more data + viz on this. i think being able to animate queries and explore them will give us a much better way to solve the problem.

i'm reminded yet again that packet switching replaced line switching.

@whyrusleeping
Copy link
Member

@jbenet its mostly dialing stuff that takes lots of time. I was going to work on some little tools for requesting records, giving the ability to specify how many to wait for and other such tweaks.

@krl
Copy link
Contributor Author

krl commented Oct 12, 2015

If a quickfix would be possible, that would be great. the --local arg as
you suggested would work. I'm blocked on this re starlog at the
moment.

@whyrusleeping
Copy link
Member

@krl i will try and get to that ASAP.

@whyrusleeping whyrusleeping self-assigned this Oct 12, 2015
@whyrusleeping whyrusleeping added kind/bug A bug in existing code (including security flaws) topic/ipns Topic ipns labels Oct 12, 2015
@whyrusleeping
Copy link
Member

@krl #1834

@jbenet
Copy link
Member

jbenet commented Oct 14, 2015

#1834 is merged.

@jbenet jbenet closed this as completed Oct 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug A bug in existing code (including security flaws) topic/ipns Topic ipns
Projects
None yet
Development

No branches or pull requests

3 participants