New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding Custom Poolsize for WASM Functions #87
Conversation
Important Auto Review SkippedReview was skipped due to path filters WalkthroughThe recent update introduces a Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChat with CodeRabbit Bot (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 2
Configuration used: CodeRabbit UI
Files ignored due to filter (1)
- testdata/tarmac.json
Files selected for processing (5)
- docs/running-tarmac/configuration.md (1 hunks)
- docs/wasm-functions/multi-function-services.md (2 hunks)
- pkg/app/app.go (2 hunks)
- pkg/app/server_test.go (2 hunks)
- pkg/config/config.go (1 hunks)
Files skipped from review due to trivial changes (1)
- pkg/app/server_test.go
Additional comments: 4
docs/running-tarmac/configuration.md (1)
- 29-29: The addition of
APP_WASM_POOL_SIZE
to the configuration options is consistent with the existing table format and aligns with the PR objectives to allow users to configure a custom pool size for WASM functions.pkg/config/config.go (1)
- 43-44: The addition of the
PoolSize
field to theFunction
struct is consistent with the PR objectives to allow configuration of the number of WASM instances and aligns with the AI-generated summary.docs/wasm-functions/multi-function-services.md (2)
20-21: The documentation update to include the
pool_size
property in thetarmac.json
example is consistent with the PR objectives and the AI-generated summary.65-65: The update to the description of the function object, replacing
configuration
withpool_size
, is consistent with the PR objectives and the AI-generated summary.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #87 +/- ##
===========================================
+ Coverage 29.93% 81.42% +51.48%
===========================================
Files 12 12
Lines 1306 1308 +2
===========================================
+ Hits 391 1065 +674
+ Misses 866 181 -685
- Partials 49 62 +13 ☔ View full report in Codecov by Sentry. |
This PR adds a custom pool size configuration for both the default WASM function and multi-function configuration files. This allows users to specify a custom number of WASM instances to load into memory. Some functions (like an init function) may not need multiple instances, and some, like a request handler, might need many.
Summary by CodeRabbit
New Features
PoolSize
configuration.Documentation
pool_size
property in the configuration files.Bug Fixes