Skip to content

Conversation

@mplatt8
Copy link
Collaborator

@mplatt8 mplatt8 commented May 22, 2025

…uilt in CI to enable setrlimit syscall. linux using prlimit for increase

Marc Platt added 2 commits May 22, 2025 11:42
console.log(`Current file descriptor limit: ${currentFdLimit}`);

// Wrapper for spawn that handles file descriptor limits
function spawnWithHighLimits(command, args, options = {}) {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this function do anything?

if (this.initialized && this.nativeModule) {
// If we have the native module, assume the limit was set by it
// In the future, we could add a getrlimit call to the native module
return 1000;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like there is a call to getrlimit in set_limits.cc, can you use that here?

@mplatt8 mplatt8 merged commit 47882c3 into master May 22, 2025
4 checks passed
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.

3 participants