Skip to content

Commit

Permalink
chore: lint --fix code
Browse files Browse the repository at this point in the history
  • Loading branch information
matthewkeil committed Aug 23, 2023
1 parent d38c7a5 commit 78d0900
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ export class WorkerNetworkCore implements INetworkCore {

const worker = new Worker("./networkCoreWorker.js", {
workerData,
/**
/**
* maxYoungGenerationSizeMb defaults to 152mb through the cli option defaults.
* That default value was determined via https://github.com/ChainSafe/lodestar/issues/2115 and
* should be tuned further as needed. If we update network code and see substantial
Expand All @@ -119,7 +119,7 @@ export class WorkerNetworkCore implements INetworkCore {
* number causes detrimental slowdown from increased variable lookup time. Empirical evidence
* showed that there is a pretty big window of "correct" values but we can always tune as
* necessary
*/
*/
resourceLimits: {maxYoungGenerationSizeMb: opts.maxYoungGenerationSizeMb},
} as ConstructorParameters<typeof Worker>[1]);

Expand Down
2 changes: 1 addition & 1 deletion packages/cli/src/options/beaconNodeOptions/network.ts
Original file line number Diff line number Diff line change
Expand Up @@ -393,6 +393,6 @@ export const options: CliCommandOptions<NetworkArgs> = {
hidden: true,
group: "network",
description: "Max size of young generation in megabytes. Defaults to 152mb",
defaultDescription: String(defaultOptions.network.maxYoungGenerationSizeMb)
defaultDescription: String(defaultOptions.network.maxYoungGenerationSizeMb),
},
};
2 changes: 1 addition & 1 deletion packages/cli/test/unit/options/beaconNodeOptions.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ describe("options / beaconNodeOptions", () => {
rateLimitMultiplier: 1,
maxGossipTopicConcurrency: 64,
useWorker: true,
maxYoungGenerationSizeMb: 152
maxYoungGenerationSizeMb: 152,
},
sync: {
isSingleNode: true,
Expand Down

0 comments on commit 78d0900

Please sign in to comment.