fix: getting timeout exceeded in the sdk client even when transaction is submitted #3129

konstantinabl · 2024-10-18T15:27:59Z

Description:

Adds improved logging for more visibility about the nodes we are querying during sendRawTransaction

Related issue(s):

github-actions · 2024-10-18T15:31:45Z

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled validates enforcement of request id. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 27,178.6 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 1.46 MB
Total Heap Size Executable: no changes
Total Physical Size: decreased with 348.16 KB
Total Available Size: decreased with 8.23 MB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 3.48 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.84 MB
- Space Used Size: increased with 2.04 MB
- Space Available Size: decreased with 11.04 MB
- Physical Space Size: increased with 1.84 MB
Large Object Space:
- Space Size: increased with 835.58 KB
- Space Used Size: increased with 813.50 KB
- Space Available Size: no changes
- Physical Space Size: increased with 835.58 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions · 2024-10-18T15:32:09Z

Tests

      3 files   398 suites 19s ⏱️
1 417 tests 1 416 ✔️ 1 💤 0 ❌
1 426 runs 1 425 ✔️ 1 💤 0 ❌

Results for commit c75181a.

♻️ This comment has been updated with latest results.

github-actions · 2024-10-18T15:46:50Z

Acceptance Tests

  17 files 256 suites 29m 5s ⏱️
524 tests 516 ✔️ 4 💤 4 ❌
691 runs 683 ✔️ 4 💤 4 ❌

Results for commit c75181a.

♻️ This comment has been updated with latest results.

victor-yanev · 2024-10-22T08:21:36Z

LGTM, but the name of this PR is kind of misleading

github-actions · 2024-10-23T13:07:18Z

Test Results

21 files + 2 284 suites +14 37m 59s ⏱️ + 2m 37s
606 tests - 2 595 ✅ - 1 4 💤 ±0 7 ❌ - 1
747 runs +51 730 ✅ +46 5 💤 +1 12 ❌ +4

For more details on these failures, see this check.

Results for commit 2158bc7. ± Comparison against base commit 3979ee0.

This pull request removes 4 and adds 2 tests. Note that renamed tests count towards both.

"after each" hook for "@release captures transfer events" ‑ RPC Server Acceptance Tests Acceptance tests @web-socket-batch-3 eth_subscribe "after each" hook for "@release captures transfer events"
"before all" hook for "Subscribes for debug" ‑ RPC Server Acceptance Tests Acceptance tests @web-socket-batch-3 eth_subscribe Subscribes to log events "before all" hook for "Subscribes for debug"
"before all" hook in "@web-socket-batch-3 eth_subscribe newHeads" ‑ RPC Server Acceptance Tests Acceptance tests @web-socket-batch-3 eth_subscribe newHeads "before all" hook in "@web-socket-batch-3 eth_subscribe newHeads"
"before each" hook for "@release captures transfer events" ‑ RPC Server Acceptance Tests Acceptance tests @web-socket-batch-3 eth_subscribe "before each" hook for "@release captures transfer events"

"before all" hook in "@api-batch-2 RPC Server Acceptance Tests" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests "before all" hook in "@api-batch-2 RPC Server Acceptance Tests"
"before each" hook for "should execute "eth_getStorageAt" request to get old state with passing specific block" ‑ RPC Server Acceptance Tests Acceptance tests @api-batch-2 RPC Server Acceptance Tests "before each" hook for "should execute "eth_getStorageAt" request to get old state with passing specific block"

♻️ This comment has been updated with latest results.

github-actions · 2024-10-23T13:18:32Z

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should execute "eth_accounts". This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 28,942 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 262.14 KB
Total Heap Size Executable: no changes
Total Physical Size: decreased with 450.56 KB
Total Available Size: decreased with 633.34 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 192.00 bytes
Used Heap Size: decreased with 770.06 KB
Heap Size Limit: no changes
Malloced Memory: no changes
External Memory: decreased with 17.95 KB
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

Old Space:
- Space Size: increased with 262.14 KB
- Space Used Size: increased with 105.20 KB
- Space Available Size: decreased with 1.00 MB
- Physical Space Size: increased with 262.14 KB

GC Type: MarkSweepCompact
Cost: 38,546.4 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 270.34 KB
Total Available Size: decreased with 1.16 MB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 96.00 bytes
Used Heap Size: decreased with 378.25 KB
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):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: no changes
- Space Available Size: no changes
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions · 2024-10-23T13:18:35Z

🚨 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: Scavenge
Cost: 5,893.37 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 262.14 KB
Total Heap Size Executable: no changes
Total Physical Size: decreased with 253.95 KB
Total Available Size: increased with 269.04 KB
Total Global Handles Size: no changes
Used Global Handles Size: no changes
Used Heap Size: decreased with 276.60 KB
Heap Size Limit: no changes
Malloced Memory: no changes
External Memory: decreased with 165.00 bytes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

Old Space:
- Space Size: increased with 262.14 KB
- Space Used Size: increased with 335.65 KB
- Space Available Size: decreased with 81.06 KB
- Physical Space Size: increased with 262.14 KB

GC Type: Scavenge
Cost: 4,363.62 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 1.31 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 794.62 KB
Total Available Size: decreased with 817.10 KB
Total Global Handles Size: no changes
Used Global Handles Size: no changes
Used Heap Size: decreased with 237.34 KB
Heap Size Limit: no changes
Malloced Memory: decreased with 1.16 MB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 571.23 KB
- Space Available Size: increased with 571.23 KB
- Physical Space Size: increased with 532.48 KB
Old Space:
- Space Size: increased with 262.14 KB
- Space Used Size: increased with 333.90 KB
- Space Available Size: decreased with 77.62 KB
- Physical Space Size: increased with 262.14 KB

GC Type: MarkSweepCompact
Cost: 31,849.3 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 274.43 KB
Total Available Size: decreased with 1.69 MB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 96.00 bytes
Used Heap Size: decreased with 1.18 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):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 429.19 KB
- Space Available Size: increased with 429.19 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions · 2024-10-23T13:18:59Z

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should execute "net_peerCount". This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 24,689.8 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 274.43 KB
Total Available Size: decreased with 392.87 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 850.66 KB
Heap Size Limit: no changes
Malloced Memory: decreased with 49.44 KB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 576.50 KB
- Space Available Size: increased with 576.50 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions · 2024-10-23T13:19:00Z

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should execute "eth_sendTransaction". This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 29,612.8 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.10 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 32.77 KB
Total Available Size: decreased with 1.33 MB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 732.97 KB
Heap Size Limit: no changes
Malloced Memory: increased with 32.78 KB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 579.31 KB
- Space Available Size: increased with 1.61 MB
- Physical Space Size: increased with 32.77 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions · 2024-10-23T13:19:24Z

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should not execute batch request when disabled. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 36,691.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 274.43 KB
Total Available Size: decreased with 1.12 MB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 819.68 KB
Heap Size Limit: no changes
Malloced Memory: increased with 59.75 KB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 362.80 KB
- Space Available Size: increased with 362.80 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions · 2024-10-23T13:21:07Z

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should execute with empty TracerConfigWrapper.tracerConfig. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 30,414 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 274.43 KB
Total Available Size: decreased with 715.16 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 699.95 KB
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):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 467.07 KB
- Space Available Size: increased with 467.07 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions · 2024-10-23T13:21:08Z

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should fail with valid hash and invalid TracerType string. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 29,594 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.36 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 294.91 KB
Total Available Size: decreased with 975.68 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 64.00 bytes
Used Heap Size: decreased with 788.81 KB
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):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 669.28 KB
- Space Available Size: increased with 1.70 MB
- Physical Space Size: increased with 32.77 KB
Old Space:
- Space Size: increased with 262.14 KB
- Space Used Size: decreased with 21.21 KB
- Space Available Size: decreased with 1.33 MB
- Physical Space Size: increased with 262.14 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions · 2024-10-23T13:21:31Z

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled throws an error if address is NOT 0x prefixed. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 27,052.8 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 274.43 KB
Total Available Size: decreased with 970.70 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 32.00 bytes
Used Heap Size: decreased with 88.88 KB
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):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 64.13 KB
- Space Available Size: increased with 64.13 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions · 2024-10-23T13:21:33Z

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled throws an error if the param is not an array. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 26,363.8 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.10 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 32.77 KB
Total Available Size: decreased with 407.15 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 32.00 bytes
Used Heap Size: decreased with 149.97 KB
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):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 114.80 KB
- Space Available Size: increased with 1.15 MB
- Physical Space Size: increased with 32.77 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions · 2024-10-23T13:21:57Z

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled throws error if block number is not correct tag. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 49,420.3 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 274.43 KB
Total Available Size: decreased with 760.46 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 32.00 bytes
Used Heap Size: decreased with 291.63 KB
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):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 64.05 KB
- Space Available Size: increased with 64.05 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions · 2024-10-23T13:22:01Z

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled throws an error for input: {"tracer":"opcodeLogger","tracerConfig":{"disableStack":"invalid"}}. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 71,252.1 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.10 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 32.77 KB
Total Available Size: increased with 275.46 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 32.00 bytes
Used Heap Size: decreased with 296.98 KB
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):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 67.71 KB
- Space Available Size: increased with 1.10 MB
- Physical Space Size: increased with 32.77 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions · 2024-10-23T13:22:29Z

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled does not throw an error for input: {"tracer":"opcodeLogger","tracerConfig":{}}. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 27,844.8 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 274.43 KB
Total Available Size: decreased with 866.71 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 32.00 bytes
Used Heap Size: decreased with 182.86 KB
Heap Size Limit: no changes
Malloced Memory: decreased with 257.22 KB
External Memory: no changes
Peak Malloced Memory: no changes

Heap Space Statistics (before vs after executing the test):

New Space:
- Space Size: increased with 1.05 MB
- Space Used Size: decreased with 54.32 KB
- Space Available Size: increased with 54.32 KB
- Physical Space Size: increased with 274.43 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

github-actions · 2024-10-23T13:22:31Z

🚨 Memory Leak Detected 🚨

A potential memory leak has been detected in the test titled should return the limit duration from environment variable. This may impact the application's performance and stability.

Details

📊 Memory Leak Detection Report 📊

GC Type: MarkSweepCompact
Cost: 27,924.2 ms

Heap Statistics (before vs after executing the test):

Total Heap Size: increased with 2.10 MB
Total Heap Size Executable: no changes
Total Physical Size: increased with 32.77 KB
Total Available Size: increased with 85.82 KB
Total Global Handles Size: no changes
Used Global Handles Size: decreased with 32.00 bytes
Used Heap Size: decreased with 99.18 KB
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):

New Space:
- Space Size: increased with 2.10 MB
- Space Used Size: decreased with 80.33 KB
- Space Available Size: increased with 1.11 MB
- Physical Space Size: increased with 32.77 KB

Recommendations

Please investigate the memory allocations in this test, focusing on objects that are not being properly deallocated.

Signed-off-by: Konstantina Blazhukova <[email protected]>

sonarcloud · 2024-10-30T11:29:21Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

codecov · 2024-10-30T11:33:23Z

Codecov Report

Attention: Patch coverage is 88.88889% with 2 lines in your changes missing coverage. Please review.

Project coverage is 83.38%. Comparing base (5ca6f63) to head (2158bc7).
Report is 7 commits behind head on main.

Files with missing lines	Patch %	Lines
packages/relay/src/lib/clients/sdkClient.ts	66.66%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3129      +/-   ##
==========================================
+ Coverage   83.26%   83.38%   +0.12%     
==========================================
  Files          66       66              
  Lines        4254     4292      +38     
  Branches      829      835       +6     
==========================================
+ Hits         3542     3579      +37     
- Misses        471      473       +2     
+ Partials      241      240       -1

Flag	Coverage Δ
config-service	`98.14% <ø> (ø)`
relay	`85.58% <88.88%> (+0.13%)`	⬆️
server	`83.52% <ø> (ø)`
ws-server	`36.87% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...ages/relay/src/lib/errors/MirrorNodeClientError.ts	`91.66% <100.00%> (ø)`
packages/relay/src/lib/errors/SDKClientError.ts	`76.19% <100.00%> (ø)`
packages/relay/src/lib/eth.ts	`82.33% <100.00%> (+0.12%)`	⬆️
packages/relay/src/lib/model.ts	`83.50% <100.00%> (ø)`
packages/relay/src/lib/net.ts	`100.00% <ø> (ø)`
packages/relay/src/lib/relay.ts	`88.57% <100.00%> (+0.16%)`	⬆️
.../relay/src/lib/services/hapiService/hapiService.ts	`78.94% <100.00%> (+0.68%)`	⬆️
...s/relay/src/lib/services/hbarLimitService/index.ts	`96.66% <100.00%> (ø)`
packages/relay/src/lib/web3.ts	`100.00% <ø> (ø)`
packages/relay/src/utils.ts	`100.00% <100.00%> (ø)`
... and 1 more

... and 4 files with indirect coverage changes

konstantinabl linked an issue Oct 18, 2024 that may be closed by this pull request

Getting timeout exceeded in the sdk client even when transaction is submitted #3118

Open

konstantinabl force-pushed the 3118-getting-timeout-exceeded-in-the-sdk-client-even-when-transaction-is-submitted branch 4 times, most recently from 5196d1d to 92f5b6a Compare October 21, 2024 14:32

konstantinabl marked this pull request as ready for review October 21, 2024 14:33

konstantinabl requested review from AlfredoG87, ebadiere, lukelee-sl and a team as code owners October 21, 2024 14:33

konstantinabl changed the title ~~getting timeout exceeded in the sdk client even when transaction is submitted~~ fix: getting timeout exceeded in the sdk client even when transaction is submitted Oct 21, 2024

victor-yanev previously approved these changes Oct 22, 2024

View reviewed changes

konstantinabl marked this pull request as draft October 22, 2024 09:04

konstantinabl dismissed victor-yanev’s stale review via eb7afd5 October 22, 2024 09:55

konstantinabl force-pushed the 3118-getting-timeout-exceeded-in-the-sdk-client-even-when-transaction-is-submitted branch 5 times, most recently from 23cd57c to 8a3aaec Compare October 23, 2024 13:01

konstantinabl force-pushed the 3118-getting-timeout-exceeded-in-the-sdk-client-even-when-transaction-is-submitted branch from 8a3aaec to 913f577 Compare October 23, 2024 13:13

konstantinabl force-pushed the 3118-getting-timeout-exceeded-in-the-sdk-client-even-when-transaction-is-submitted branch from 3e5de48 to 913f577 Compare October 24, 2024 16:14

konstantinabl added 13 commits October 25, 2024 16:55

improves logging to try and trace issue

b81d899

Signed-off-by: Konstantina Blazhukova <[email protected]>

Improves logging further

f2caac7

Signed-off-by: Konstantina Blazhukova <[email protected]>

adds check for transaction in mirror node

ca4e6f8

Signed-off-by: Konstantina Blazhukova <[email protected]>

Fixes error handling

508456c

Signed-off-by: Konstantina Blazhukova <[email protected]>

Only executes when response is null

8578950

Signed-off-by: Konstantina Blazhukova <[email protected]>

Experiments to see if transaction could be found in mirrornode

90c745e

Signed-off-by: Konstantina Blazhukova <[email protected]>

Removes querying to mirrorNode

d24a51b

Signed-off-by: Konstantina Blazhukova <[email protected]>

Logs the transaction object

8f22211

Signed-off-by: Konstantina Blazhukova <[email protected]>

Adds logger to client

fc64edd

Signed-off-by: Konstantina Blazhukova <[email protected]>

Adds logging for the transaction count

33666e1

Signed-off-by: Konstantina Blazhukova <[email protected]>

Replaces sdk with own version with more logging

3e62a31

Signed-off-by: Konstantina Blazhukova <[email protected]>

Bumps timeout on sdk

858e8fe

Signed-off-by: Konstantina Blazhukova <[email protected]>

revert timeout to 10 seconds

f1dd35e

Signed-off-by: Konstantina Blazhukova <[email protected]>

konstantinabl force-pushed the 3118-getting-timeout-exceeded-in-the-sdk-client-even-when-transaction-is-submitted branch from 8a94504 to f1dd35e Compare October 25, 2024 13:58

konstantinabl added 2 commits October 25, 2024 16:58

Revert timeout to default

5585dbd

Signed-off-by: Konstantina Blazhukova <[email protected]>

remo forgotten only

b84bb7b

Signed-off-by: Konstantina Blazhukova <[email protected]>

konstantinabl mentioned this pull request Oct 28, 2024

SDK throws a timeout exceeded but the transaction can be found in hash scan hashgraph/hedera-sdk-js#2614

Open

Adds more logging

2158bc7

Signed-off-by: Konstantina Blazhukova <[email protected]>

konstantinabl force-pushed the 3118-getting-timeout-exceeded-in-the-sdk-client-even-when-transaction-is-submitted branch from 021a800 to 2158bc7 Compare October 30, 2024 11:28

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: getting timeout exceeded in the sdk client even when transaction is submitted #3129

fix: getting timeout exceeded in the sdk client even when transaction is submitted #3129

konstantinabl commented Oct 18, 2024 •

edited

Loading

github-actions bot commented Oct 18, 2024 •

edited

Loading

github-actions bot commented Oct 18, 2024 •

edited

Loading

github-actions bot commented Oct 18, 2024 •

edited

Loading

victor-yanev commented Oct 22, 2024

github-actions bot commented Oct 23, 2024 •

edited

Loading

github-actions bot commented Oct 23, 2024

github-actions bot commented Oct 23, 2024

github-actions bot commented Oct 23, 2024

github-actions bot commented Oct 23, 2024

github-actions bot commented Oct 23, 2024

github-actions bot commented Oct 23, 2024

github-actions bot commented Oct 23, 2024

github-actions bot commented Oct 23, 2024

github-actions bot commented Oct 23, 2024

github-actions bot commented Oct 23, 2024

github-actions bot commented Oct 23, 2024

github-actions bot commented Oct 23, 2024

github-actions bot commented Oct 23, 2024

sonarcloud bot commented Oct 30, 2024

codecov bot commented Oct 30, 2024

fix: getting timeout exceeded in the sdk client even when transaction is submitted #3129

Are you sure you want to change the base?

fix: getting timeout exceeded in the sdk client even when transaction is submitted #3129

Conversation

konstantinabl commented Oct 18, 2024 • edited Loading

github-actions bot commented Oct 18, 2024 • edited Loading

🚨 Memory Leak Detected 🚨

Details

Recommendations

github-actions bot commented Oct 18, 2024 • edited Loading

Tests

github-actions bot commented Oct 18, 2024 • edited Loading

Acceptance Tests

victor-yanev commented Oct 22, 2024

github-actions bot commented Oct 23, 2024 • edited Loading

Test Results

github-actions bot commented Oct 23, 2024

🚨 Memory Leak Detected 🚨

Details

Recommendations

github-actions bot commented Oct 23, 2024

🚨 Memory Leak Detected 🚨

Details

Recommendations

github-actions bot commented Oct 23, 2024

🚨 Memory Leak Detected 🚨

Details

Recommendations

github-actions bot commented Oct 23, 2024

🚨 Memory Leak Detected 🚨

Details

Recommendations

github-actions bot commented Oct 23, 2024

🚨 Memory Leak Detected 🚨

Details

Recommendations

github-actions bot commented Oct 23, 2024

🚨 Memory Leak Detected 🚨

Details

Recommendations

github-actions bot commented Oct 23, 2024

🚨 Memory Leak Detected 🚨

Details

Recommendations

github-actions bot commented Oct 23, 2024

🚨 Memory Leak Detected 🚨

Details

Recommendations

github-actions bot commented Oct 23, 2024

🚨 Memory Leak Detected 🚨

Details

Recommendations

github-actions bot commented Oct 23, 2024

🚨 Memory Leak Detected 🚨

Details

Recommendations

github-actions bot commented Oct 23, 2024

🚨 Memory Leak Detected 🚨

Details

Recommendations

github-actions bot commented Oct 23, 2024

🚨 Memory Leak Detected 🚨

Details

Recommendations

github-actions bot commented Oct 23, 2024

🚨 Memory Leak Detected 🚨

Details

Recommendations

sonarcloud bot commented Oct 30, 2024

Quality Gate passed

codecov bot commented Oct 30, 2024

Codecov Report

konstantinabl commented Oct 18, 2024 •

edited

Loading

github-actions bot commented Oct 18, 2024 •

edited

Loading

github-actions bot commented Oct 18, 2024 •

edited

Loading

github-actions bot commented Oct 18, 2024 •

edited

Loading

github-actions bot commented Oct 23, 2024 •

edited

Loading