Skip to content

Commit

Permalink
(enh) Change occ background-job:list limit default 10->100
Browse files Browse the repository at this point in the history
The default limit of 10 seems too conservative. Nearly all environments >100 and most <10. At the same time, this higher default limit still remains reasonable to avoid problems in really big environments.

Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
  • Loading branch information
joshtrichards authored and mejo- committed Nov 14, 2023
1 parent 281ac50 commit 34211c3
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion core/Command/Background/ListCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ protected function configure(): void {
'l',
InputOption::VALUE_OPTIONAL,
'Number of jobs to retrieve',
'10'
'100'
)->addOption(
'offset',
'o',
Expand All @@ -67,6 +67,7 @@ protected function configure(): void {

protected function execute(InputInterface $input, OutputInterface $output): int {
$jobs = $this->jobList->getJobsIterator($input->getOption('class'), (int)$input->getOption('limit'), (int)$input->getOption('offset'));
$output->writeln("Output is currently limited to first " . $input->getOption('limit') . " jobs. Specify `-l, --limit[=LIMIT]` to override.");
$this->writeTableInOutputFormat($input, $output, $this->formatJobs($jobs));
return 0;
}
Expand Down

0 comments on commit 34211c3

Please sign in to comment.