github-actions[bot]
commented
1 month ago 🚨 Memory Leak Detected 🚨
A potential memory leak has been detected in the test titled should execute "eth_getTransactionByHash with missing transaction". This may impact the application's performance and stability.
Details
📊 Memory Leak Detection Report 📊
GC Type: MarkSweepCompact Cost: 27,761.2 ms
Heap Statistics (before vs after executing the test):
- Total Heap Size: increased with 1.05 MB
- Total Heap Size Executable: no changes
- Total Physical Size: increased with 1.05 MB
- Total Available Size: decreased with 13.76 MB
- Total Global Handles Size: no changes
- Used Global Handles Size: decreased with 128.00 bytes
- Used Heap Size: decreased with 2.53 MB
- Heap Size Limit: no changes
- Malloced Memory: no changes
- External Memory: no changes
- Peak Malloced Memory: no changes
Heap Space Statistics (before vs after executing the test):
- Old Space:
- Space Size: increased with 1.05 MB
- Space Used Size: increased with 594.50 KB
- Space Available Size: decreased with 14.30 MB
- Physical Space Size: increased with 1.05 MB
Recommendations
Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.
sonarcloud[bot]
victor-yanev
codecov[bot]
Description:
MirrorNodeClient:getAccountmethod to optionally accept aITimestampParamsparameterprepareLogsParams,getContractResultsLogsandgetContractResultsLogsByAddresswith strong typing on theparamsobject that they accept - it now uses theIContractLogsResultsParamsinterface and this helps us track exactly where those methods are called and with which params passed to themeth_getBlockByHashandeth_getBlockByNumberto pass a timestamp range from the fetched block to subsequent calls togetLogsWithParamsRelated issue(s):
Fixes #3040
Notes for reviewer:
Checklist