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

Indexing is very slow #250

Closed
shachaf opened this issue Jan 19, 2013 · 4 comments
Closed

Indexing is very slow #250

shachaf opened this issue Jan 19, 2013 · 4 comments

Comments

@shachaf
Copy link
Collaborator

shachaf commented Jan 19, 2013

Indexing can have a lot of overhead (30x in one pseudo-benchmark) even when we don't use the Int index. Ideally it would be almost free for right-biased traversals.

@ghost ghost assigned shachaf Jan 19, 2013
@ekmett
Copy link
Owner

ekmett commented Jul 12, 2013

Is this solved in the presence of the new conjoined machinery?

@shachaf
Copy link
Collaborator Author

shachaf commented Jul 12, 2013

Well, conjoined makes it so you don't pay for Indexing if you don't use it, which is good. But I assume that it's still as slow as before if you do.

@shachaf
Copy link
Collaborator Author

shachaf commented Jul 12, 2013

I just tested it again and even when you do use indexing it looks like the performance is much better than I remembered -- maybe ~2x rather than ~30x, though I didn't try very hard. No idea what happened in the last six months.

@shachaf
Copy link
Collaborator Author

shachaf commented Jul 12, 2013

I still think we could generate better Core but I'm not sure how. We can open this issue again if we find a place where Indexing makes a big difference.

@shachaf shachaf closed this as completed Jul 12, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants