feat: add nrgorm apm integration#994
Conversation
|
jerry.wang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
7bd2c88 to
e1a1229
Compare
e1a1229 to
ec7cc0e
Compare
|
Thank you for contributing your PR. Please note that we can't proceed until the contributor license agreement has been signed. We'll also need to review the feature request for consideration to add the integration to the APM product and may potentially make further modifications as may be necessary to fully integrate it to the agent's architectural design as recommended by the review, assuming all of the above is approved. |
|
@XiXiangFiles Would it be possible to add some unit tests to this PR? |
|
Hey @XiXiangFiles This PR has aged a bit. Checking in to see if you had any uncommited changes (unit testing) we can in so we can give this a merge. Thanks! |
|
@mirackara Unit tests have been added. Could you help take a look? |
Links
#993
Details
I use the Gorm plugin to achieve the desired APM effect.