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.

Sign up for GitHub

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

Jump to bottom

[Distributed Index] Fix kvChangeIndex DocChanged() throughput bottleneck #1140

Closed
tleyden opened this issue Sep 17, 2015 · 2 comments
Closed

[Distributed Index] Fix kvChangeIndex DocChanged() throughput bottleneck #1140

tleyden opened this issue Sep 17, 2015 · 2 comments
Assignees
@adamcfraser
Labels
enhancement
Milestone
1.2.0

Comments

@tleyden
Copy link
Contributor

tleyden commented Sep 17, 2015

The fix for #1139 was just a quick workaround, which introduced a throughput bottleneck in order to achieve correct behavior. I'm adding this ticket to track the deeper fix which will take more thinking about design.
@tleyden tleyden added this to the 1.2.0 milestone Sep 17, 2015
@tleyden
Copy link
Contributor Author

tleyden commented Sep 18, 2015

One idea: maintain pool of worker goroutines for processing DocChanged callbacks

@tleyden tleyden changed the title [Distributed Index] kvChangeIndex should maintain pool of worker goroutines for processing DocChanged callbacks [Distributed Index] Fix kvChangeIndex DocChanged() throughput bottleneck Sep 18, 2015
@zgramana
Copy link
Contributor

zgramana commented Oct 2, 2015

@adamcfraser to udpate and review with @tleyden.

@zgramana zgramana added the chore label Oct 2, 2015
@zgramana zgramana added backlog and removed review labels Oct 23, 2015
@adamcfraser adamcfraser mentioned this issue Oct 27, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adamcfraser adamcfraser

Labels
enhancement
Projects
None yet
Milestone
1.2.0
Development

No branches or pull requests
3 participants
@tleyden @zgramana @adamcfraser