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

prefetch more values for jobs/list #441

Merged
merged 2 commits into from
Oct 12, 2015
Merged

prefetch more values for jobs/list #441

merged 2 commits into from
Oct 12, 2015

Conversation

coolo
Copy link
Contributor

@coolo coolo commented Oct 7, 2015

there is just too much going on

note though that this is not the original problem I was hunting :)

@coolo coolo force-pushed the speedup_list_jobs branch from 172fbad to bdf9afc Compare October 7, 2015 11:01
@@ -1,4 +1,4 @@
# Copyright (C) 2015 SUSE Linux GmbH
# Copyright (C) 2015 SUSE LLC
Copy link
Contributor

Choose a reason for hiding this comment

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

In the mail from legal they suggest not to overwrite/delete old (c) rather just append new one.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

But I don't like that

there is just too much going on
@coolo coolo force-pushed the speedup_list_jobs branch from bdf9afc to 9b09c0a Compare October 9, 2015 12:01
@aaannz
Copy link
Contributor

aaannz commented Oct 12, 2015

LGTM

coolo added a commit that referenced this pull request Oct 12, 2015
prefetch more values for jobs/list
@coolo coolo merged commit 5a69727 into master Oct 12, 2015
@coolo coolo deleted the speedup_list_jobs branch October 12, 2015 12:27
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