ecoppens/colmena
1
0
Fork 0
forked from DGNum/colmena
Code Pull requests Activity

apply: Disable configuration of build process limit

Browse source
This commit is contained in:
Zhaofeng Li 2021-02-17 08:09:15 -08:00
parent ee52032e4c
commit 9eae937b42
2 changed files with 2 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/command/apply.rs
View file

@ -192,13 +192,6 @@ pub async fn run(_global_args: &ArgMatches<'_>, local_args: &ArgMatches<'_>) {
local_args.value_of("parallel").unwrap().parse::<usize>().unwrap()
}
});
parallelism_limit.set_build_limit({
let limit = local_args.value_of("parallel").unwrap().parse::<usize>().unwrap();
if limit == 0 {
panic!("The build parallelism limit must not be 0");
}
limit
});
deployment.set_parallelism_limit(parallelism_limit);
let evaluation_node_limit = match local_args.value_of("eval-node-limit").unwrap() {

4
src/nix/deployment.rs
View file

@ -552,7 +552,7 @@ impl Default for ParallelismLimit {
}
impl ParallelismLimit {
// Do we actually want this to be configurable?
// Do we actually want them to be configurable?
/*
/// Sets the concurrent evaluation limit.
///
@ -562,12 +562,12 @@ impl ParallelismLimit {
pub fn set_evaluation_limit(&mut self, limit: usize) {
self.evaluation = Semaphore::new(limit);
}
*/
/// Sets the concurrent build limit.
pub fn set_build_limit(&mut self, limit: usize) {
self.build = Semaphore::new(limit);
}
*/
/// Sets the concurrent apply limit.
pub fn set_apply_limit(&mut self, limit: usize) {