Skip to content

Put the ffmpeg process in a setTimeout#126

Open
karimmakhloufi wants to merge 1 commit intoJamesKyburz:mainfrom
karimmakhloufi:patch-1
Open

Put the ffmpeg process in a setTimeout#126
karimmakhloufi wants to merge 1 commit intoJamesKyburz:mainfrom
karimmakhloufi:patch-1

Conversation

@karimmakhloufi
Copy link
Copy Markdown

To avoid the maximum call stack range error when multiple simultaneous calls are made or even when used in a cue with only 1 worker.

I did time it and a 1h30 video and did not see any performance drop

karim@karim-laptop  ~/Work/streamtube/xp  time node index.js
node index.js 52,42s user 3,99s system 112% cpu 50,100 total
karim@karim-laptop  ~/Work/streamtube/xp  time node index.js
node index.js 52,34s user 4,13s system 115% cpu 48,956 total

To avoid the maximum call stack range error when multiple simultaneous calls are made or even when used in a cue with only 1 worker.

I did time it and a 1h30 video and did not see any performance drop

 karim@karim-laptop  ~/Work/streamtube/xp  time node index.js 
node index.js  52,42s user 3,99s system 112% cpu 50,100 total
 karim@karim-laptop  ~/Work/streamtube/xp  time node index.js
node index.js  52,34s user 4,13s system 115% cpu 48,956 total
@tbkunboy
Copy link
Copy Markdown

Have you tested on the real server yet. I try ec2 aws amazon build and it often stops. Is there a way to fix it.

@karimmakhloufi
Copy link
Copy Markdown
Author

I've only tried it on heroku and on my laptop for now, I'll try on ec2 to reproduce.

@JamesKyburz JamesKyburz force-pushed the main branch 3 times, most recently from 62c3a4e to 0199a9d Compare August 28, 2021 17:24
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