Skip to content
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

merge development to main #1430

Merged
merged 25 commits into from
Jan 15, 2025
Merged

merge development to main #1430

merged 25 commits into from
Jan 15, 2025

Conversation

dragos-rebegea
Copy link
Contributor

No description provided.

bogdan-rosianu and others added 23 commits November 28, 2024 11:20
…igger

Update load test workflow triggers
* EGLD-000000: add EGLD-000000 support

* EGLD-000000: add EGLD-000000 support

* filter out egld-000000 from all tokens response + tests fixes

* refacotring

---------

Co-authored-by: bogdan-rosianu <[email protected]>
…rked-as-can-be-ignored

API-339: include transactions marked as canBeIgnored
…nder

# Conflicts:
#	src/endpoints/transactions/entities/transaction.filter.ts
API-334: add option to include relayed sc results
…-simulate

forward checkSignature param for tx simulation
…sses (#1374)

* Add 'addresses' query parameter to filter accounts by a list of addresses

* Add bulk account retrieval method and integrate with account service

* Update request type in getAccountsBulk method to use addressesBulk

* Add 'addressesBulk' request type to GatewayComponentRequest enum
* remove new keyword for query parsers without specific parameter

* remove left new keywords

* remove new keyword for ParseArrayPipe s that have no parameter
…o data is returned, ensuring a default response of remainingUnBondPeriod as 0. (#1427)
* Enhance ApiMetricsService with new transaction and batch update counters.

* Updated WebSocketPublisherController to emit metrics events after processing transactions and batch updates.

* Update web.socket controller spec
* added function name for pool transactions

* fix modules imports

* fix unit tests

* add function filter
* accounts endpoint make es heavy fiealds optional

* optimize provider query for simple accounts

* early return for null account

* use AccountFetchOptions

* fixes after review

* fixes after review

* Enhance AccountController tests to validate optional parameters in account retrieval. Added tests for withTxCount, withScrCount, withTimestamp, and withAssets parameters, ensuring correct behavior and response structure. Updated existing tests to reflect changes in expected account details when optional parameters are used. Improved overall test coverage for account details retrieval.

---------

Co-authored-by: bogdan-rosianu <[email protected]>
Co-authored-by: cfaur09 <[email protected]>
Copy link

github-actions bot commented Jan 10, 2025

k6 load testing comparison.
Base Commit Hash: dbec8bd
Target Commit Hash: c38c4fa

Metric Base Target Diff
AvgMax9095AvgMax9095AvgMax9095
Nodes56.38660.5661.5964.0749.31537.0353.8155.96-12.55% ✅-18.70% ✅-12.64% ✅-12.65% ✅
Accounts56.89848.9861.5864.1350.311123.0353.8755.95-11.57% ✅+32.28% 🔴-12.52% ✅-12.76% ✅
Blocks64.081523.7662.5568.3257.601362.3954.8257.72-10.10% ✅-10.59% ✅-12.35% ✅-15.52% ✅
Transactions75.673142.1462.7367.6866.573359.0854.9857.89-12.03% ✅+6.90% 🔴-12.36% ✅-14.47% ✅
Mex56.02380.7461.6564.1648.96421.7253.8055.86-12.61% ✅+10.77% 🔴-12.74% ✅-12.94% ✅
Pool55.88298.6261.5363.9649.28411.0053.8256.00-11.80% ✅+37.63% 🔴-12.53% ✅-12.44% ✅
Tokens56.28589.7861.6263.9649.67682.4553.8455.97-11.76% ✅+15.71% 🔴-12.62% ✅-12.50% ✅
Test Run Duration60003.2160002.85

Legend: Avg - Average Response Time, Max - Maximum Response Time, 90 - 90th Percentile, 95 - 95th Percentile
All times are in milliseconds.

cfaur09
cfaur09 previously approved these changes Jan 10, 2025
bogdan-rosianu
bogdan-rosianu previously approved these changes Jan 10, 2025
* added sovereign

* added placeholder config

* added placeholder config

* added placeholder config

---------

Co-authored-by: liviuancas-elrond <[email protected]>
@dragos-rebegea dragos-rebegea dismissed stale reviews from bogdan-rosianu and cfaur09 via 9aa2e7b January 13, 2025 17:00
@dragos-rebegea dragos-rebegea merged commit 914e3c9 into main Jan 15, 2025
7 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.

5 participants