Skip to content

Commit

Permalink
[teraslice-cli] clean up 'jobs export' command to run serially withou…
Browse files Browse the repository at this point in the history
…t pMap (#3936)

When running `teraslice-cli jobs export`, remove the pMap wrapper from
the call to `exportOne()` as it is useless if we set concurrency to 1.

Bump teraslice-cli from `2.10.1` to `2.10.2`

ref: #3844
  • Loading branch information
busma13 authored Jan 31, 2025
1 parent fab2b33 commit aaa27c4
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
2 changes: 1 addition & 1 deletion packages/teraslice-cli/package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "teraslice-cli",
"displayName": "Teraslice CLI",
"version": "2.10.1",
"version": "2.10.2",
"description": "Command line manager for teraslice jobs, assets, and cluster references.",
"keywords": [
"teraslice"
Expand Down
8 changes: 3 additions & 5 deletions packages/teraslice-cli/src/helpers/jobs.ts
Original file line number Diff line number Diff line change
Expand Up @@ -822,11 +822,9 @@ export default class Jobs {

reply.yellow(`Saving jobFile(s) for ${jobIds.join(', ')} on ${this.config.args.clusterAlias}`);

await pMap(
this.jobs,
(job) => this.exportOne(job.config),
{ concurrency: 1 }
);
for (const job of this.jobs) {
await this.exportOne(job.config);
}

reply.green(`Saved jobFile(s) to ${this.config.outdir}`);
}
Expand Down

0 comments on commit aaa27c4

Please sign in to comment.