AElf
AElf copied to clipboard
Add tracking for monitoring system performance
Closes #3542 Add tracking for monitoring system performance close
Pull request analysis by VIZIPI
Below you will find who is the most qualified team member to review your code. This analysis includes his/her work on the code included in this Pull request, in addition to their experience in code affected by these changes ( partly found within the list of potential missing files below ) Feedback always welcome
Reviewers with knowledge related to these changes
| Match % | Person | Commit Count | Common Files |
|---|---|---|---|
| 93.18% | EanCuznaivy | 345 | 41 |
| 70.45% | ZhangXin | 212 | 31 |
| 68.18% | jason-hoopox | 167 | 30 |
| 47.73% | eanzhao | 46 | 21 |
| 47.73% | 轩辕焮痕 | 2 | 21 |
| 45.45% | gldeng | 41 | 20 |
Potential missing files from this Pull request
files commonly committed with a subset of this pr, but not committed this time. (click to collapse)
| File | Percentile | rate |
| contract/AElf.Contracts.MultiToken/TokenContract_Fees.cs | 50.00% | 1 out of 2 times |
| contract/AElf.Contracts.MultiToken/TokenContract_Views.cs | 50.00% | 1 out of 2 times |
| contract/AElf.Contracts.MultiToken/TokenContract_Helper.cs | 50.00% | 1 out of 2 times |
| test/AElf.Contracts.TestBase/ContractTester.cs | 50.00% | 1 out of 2 times |
| ...test/AElf.WebApp.Application.Chain.Tests/BlockChainAppSer | 50.00% | 1 out of 2 times |
| ...contract/AElf.Contracts.CrossChain/CrossChainContract_Hel | 50.00% | 1 out of 2 times |
| test/AElf.OS.TestBase/OSTestHelper.cs | 50.00% | 1 out of 2 times |
| ...src/AElf.Kernel.SmartContractExecution/Application/BlockE | 50.00% | 1 out of 2 times |
| ...test/AElf.Contracts.MultiToken.Tests/MultiTokenContractTe | 50.00% | 1 out of 2 times |
| AElf.sln | 50.00% | 1 out of 2 times |
Committed file ranks
(click to expand)
96.45%[src/AElf.Launcher/Startup.cs] 51.71%[src/AElf.Kernel.Core/UmpInterceptor.cs] 92.13%[...src/AElf.Kernel.Consensus.Core/Application/ConsensusServi] 99.80%[src/AElf.Core/AElf.Core.csproj] 98.72%[src/AElf.Runtime.CSharp/Executive.cs] 74.51%[...src/AElf.Kernel.SmartContract/Application/IExecutionObser] 99.39%[...src/AElf.Kernel.SmartContract/Application/SmartContractEx] 98.61%[src/AElf.Kernel/Miner/Application/MinerService.cs] 93.90%[...src/AElf.Kernel.Core/Blockchain/Application/ITransactionB] 99.11%[...src/AElf.Kernel.SmartContract.Parallel/Application/LocalP]
Codecov Report
Attention: Patch coverage is 78.18182% with 24 lines in your changes are missing coverage. Please review.
Project coverage is 81.43%. Comparing base (
e2996bd) to head (b61c7e9).
Additional details and impacted files
@@ Coverage Diff @@
## release/1.6.0-metrics #3543 +/- ##
=========================================================
- Coverage 90.67% 81.43% -9.25%
=========================================================
Files 663 666 +3
Lines 25433 25532 +99
Branches 2283 2287 +4
=========================================================
- Hits 23061 20791 -2270
- Misses 2256 4605 +2349
- Partials 116 136 +20
| Files | Coverage Δ | |
|---|---|---|
| src/AElf.Core/CoreAElfModule.cs | 74.19% <100.00%> (+0.86%) |
:arrow_up: |
| ...nel.Consensus.Core/Application/ConsensusService.cs | 100.00% <100.00%> (ø) |
|
| ...Kernel.Core/Account/Application/IAccountService.cs | 100.00% <ø> (ø) |
|
| ....Core/Blockchain/Application/IBlockchainService.cs | 100.00% <100.00%> (ø) |
|
| ...chain/Application/ITransactionBlockIndexService.cs | 100.00% <ø> (ø) |
|
| ...lockchain/Application/ITransactionResultService.cs | 100.00% <ø> (ø) |
|
| .../Application/SystemTransactionGenerationService.cs | 100.00% <ø> (ø) |
|
| src/AElf.Kernel.Core/UmpAttribute.cs | 100.00% <100.00%> (ø) |
|
| ...cation/LocalParallelTransactionExecutingService.cs | 100.00% <100.00%> (ø) |
|
| ...martContract.Parallel/Domain/TransactionGrouper.cs | 90.00% <100.00%> (-8.53%) |
:arrow_down: |
| ... and 27 more |