Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@lucacopa
Copy link
Contributor

No description provided.

@lucacopa lucacopa force-pushed the pass_memory_estimates branch from 5eeaf12 to e8e92fe Compare June 23, 2015 20:42
@hufnagel
Copy link
Member

Won't be able to test this until next week, but go ahead and test it together with the other patches. Didn't realize this has to be forwarded from the spec through the job splitter into every single job, but given how the JobSubmitter works (only knowing job records), it makes sense.

Question, so merge/logcollect/harvest/cleanup/etc jobs, that don't do this, all get 1250 as default ?

@lucacopa
Copy link
Contributor Author

Yeah, currently the only splitting algorithms that support this are: FileBased, EventBased, LumiBased and EventAwareLumiBased. This patch would add Repack and Express to this list. All the other algorithms in WMCore or the T0 are not passing the memory requirement from the request, so they are all setting the 1250 default on the jobs.

@lucacopa
Copy link
Contributor Author

@hufnagel Patch is working for both Express and Repack jobs

hufnagel added a commit that referenced this pull request Jun 26, 2015
Pass memory estimates through the splitting algorithms
@hufnagel hufnagel merged commit 0770510 into dmwm:master Jun 26, 2015
@lucacopa lucacopa deleted the pass_memory_estimates branch June 26, 2015 00:37
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