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

Async API for CXX plugins #686

Merged
merged 2 commits into from
Oct 10, 2024
Merged

Async API for CXX plugins #686

merged 2 commits into from
Oct 10, 2024

Conversation

eeliu
Copy link
Collaborator

@eeliu eeliu commented Oct 9, 2024

Describe your changes

why we need this pr?

Issue ticket number and link

if it has issues, add them

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.
  • Do we need to implement analytics?
  • Will this be part of a product update? If yes, please write one phrase about this update.

add v0.7 version
Copy link

codecov bot commented Oct 9, 2024

Codecov Report

Attention: Patch coverage is 86.61088% with 64 lines in your changes missing coverage. Please review.

Please upload report for BASE (v0.7@c352320). Learn more about missing BASE report.

Files with missing lines Patch % Lines
common/src/common.cpp 71.15% 21 Missing and 9 partials ⚠️
common/src/NodePool/PoolManager.cpp 71.42% 2 Missing and 8 partials ⚠️
common/src/NodePool/PoolManager.h 68.96% 8 Missing and 1 partial ⚠️
common/src/ConnectionPool/TransLayer.h 0.00% 2 Missing and 2 partials ⚠️
common/src/ConnectionPool/TransLayer.cpp 25.00% 1 Missing and 2 partials ⚠️
common/src/NodePool/TraceNode.cpp 91.42% 0 Missing and 3 partials ⚠️
common/src/NodePool/test_node.cc 95.94% 2 Missing and 1 partial ⚠️
common/src/test_common.cc 98.54% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             v0.7     #686   +/-   ##
=======================================
  Coverage        ?   79.61%           
=======================================
  Files           ?       21           
  Lines           ?     1599           
  Branches        ?      170           
=======================================
  Hits            ?     1273           
  Misses          ?      191           
  Partials        ?      135           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@eeliu eeliu force-pushed the feat-asnyc-cpp branch 3 times, most recently from bc1486f to 1a86f21 Compare October 10, 2024 02:45
- pack pinpoint-common
- add cpp example
- add get depth
- remove unused code
- fixed CI
- test_pinpoint.cpp and test_pinpoint.c
- common library support spanChunk
- rename calls to fols
- add thread_local pool
- support "Async Invocation .."
- testapps/cpp example
@eeliu eeliu added this to the async api milestone Oct 10, 2024
@eeliu eeliu merged commit 5ed2744 into pinpoint-apm:v0.7 Oct 10, 2024
57 checks passed
@eeliu eeliu deleted the feat-asnyc-cpp branch October 10, 2024 05:55
eeliu added a commit to eeliu/pinpoint-c-agent that referenced this pull request Oct 10, 2024
* test pip download `pinpointPy`

* Create SECURITY.md

supported version for pinpointPy and pinpoint-php

* Create codeql.yml

* Create dependabot.yml

* Update codeql.yml

* add testapps for laravel

* set pinpoint-php-aop version

* Bump actions/download-artifact from 3 to 4.1.7 in /.github/workflows (pinpoint-apm#678)

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 3 to 4.1.7.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v3...v4.1.7)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Async API for CXX plugins (pinpoint-apm#686)

[spanChunk ] common library

- pack pinpoint-common
- add cpp example
- add get depth
- remove unused code
- fixed CI
- test_pinpoint.cpp and test_pinpoint.c
- common library support spanChunk
- rename calls to fols
- add thread_local pool
- support "Async Invocation .."
- testapps/cpp example

* fix doc and workflows

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

1 participant