Skip to content

Commit 2c204a1

Browse files
sshaderConvex, Inc.
authored andcommitted
Fix return type of ctx.scheduler.run* in node actions (#28009)
These should return the job ID but instead were returning `{ jobId: string }` GitOrigin-RevId: 82c27ac8e6fc908a702fd22e3ae25b1ede6459c1
1 parent 0fa8a6c commit 2c204a1

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

npm-packages/node-executor/src/syscalls.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -483,7 +483,7 @@ export class SyscallsImpl {
483483
scheduleSchema,
484484
operationName,
485485
);
486-
return this.actionCallback({
486+
const { jobId } = await this.actionCallback({
487487
version: scheduleArgs.version,
488488
body: {
489489
udfPath: scheduleArgs.name,
@@ -494,6 +494,7 @@ export class SyscallsImpl {
494494
operationName,
495495
responseValidator: scheduleReturn,
496496
});
497+
return jobId;
497498
}
498499

499500
async syscallCancelJob(rawArgs: string): Promise<JSONValue> {

0 commit comments

Comments
 (0)