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

[#140] Support nested async trace #153

Merged
merged 1 commit into from
Nov 22, 2023

Conversation

feelform
Copy link
Collaborator

No description provided.

@feelform feelform linked an issue Nov 21, 2023 that may be closed by this pull request
@feelform
Copy link
Collaborator Author

feelform commented Nov 21, 2023

LCOV of commit c92b2db during CI #333

Summary coverage rate:
  lines......: 82.2% (2576 of 3135 lines)
  functions..: 77.7% (518 of 667 functions)
  branches...: 68.9% (999 of 1450 branches)

Files changed coverage rate:
                                                                               |Lines       |Functions  |Branches    
  Filename                                                                     |Rate     Num|Rate    Num|Rate     Num
  ===================================================================================================================
  lib/agent.js                                                                 |83.3%     54|75.0%    12|71.4%     14
  lib/context/context-manager.js                                               |42.9%     21| 0.0%     6| 0.0%      8
  lib/instrumentation/context/local-storage.js                                 |90.9%     11| 100%     5|50.0%      2
  lib/instrumentation/instrument-method-context.js                             |83.3%      6|75.0%     4|    -      0
  lib/instrumentation/instrument-method.js                                     |93.1%     72| 100%     6|82.1%     56
  lib/instrumentation/module/express.js                                        |86.7%     15| 100%     1| 100%      2
  lib/instrumentation/module/express/express-route-interceptor.js              |90.0%     10| 100%     2|66.7%      6
  lib/instrumentation/module/express/patch-layer.js                            |90.7%     54| 100%     7|76.3%     38
  lib/instrumentation/module/koa-router.js                                     |80.0%     10| 100%     1|50.0%      2
  lib/instrumentation/module/koa/koa-register-interceptor.js                   |75.0%     40| 100%     3|62.1%     29
  lib/instrumentation/module/mysql.js                                          |88.2%     17| 100%     1|50.0%      2
  lib/instrumentation/module/mysql/mysql-cluster-of-interceptor.js             |90.0%     10| 100%     3|50.0%      2
  lib/instrumentation/module/mysql/mysql-create-connection-interceptor.js      | 100%     16| 100%     3|50.0%      2
  lib/instrumentation/module/mysql/mysql-create-pool-cluster-interceptor.js    |90.0%     10| 100%     3|50.0%      2
  lib/instrumentation/module/mysql/mysql-create-pool-interceptor.js            | 100%     14| 100%     4|50.0%      4
  lib/instrumentation/module/mysql/mysql-get-connection-interceptor.js         |90.5%     21| 100%     4|66.7%     12
  lib/instrumentation/module/mysql2.js                                         |88.2%     17| 100%     1|50.0%      2

@feelform feelform force-pushed the nested-async-trace branch 17 times, most recently from 1b2439f to a6029dd Compare November 22, 2023 08:58
* Remove no needs an argument TraceContext
* Support nested AsyncTraces
@feelform feelform merged commit ecb98fc into pinpoint-apm:master Nov 22, 2023
@feelform feelform deleted the nested-async-trace branch November 22, 2023 13:15
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.

Support nested AsyncTrace
1 participant