| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
Please update the supported-list.md about your new versions. |
Sorry, something went wrong.
There was a problem hiding this comment.
Add support for tracing Lettuce client version 6.5+ by refactoring common instrumentation code into a shared module, introducing a version-specific plugin, and providing an end-to-end test scenario.
Copilot reviewed 37 out of 38 changed files in this pull request and generated 2 comments.
Show a summary per file| File | Description |
|---|---|
| test/plugin/scenarios/lettuce-6.5.x-scenario/support-version.list | Add supported version 6.5.0.RELEASE |
| test/plugin/scenarios/lettuce-6.5.x-scenario/src/main/java/.../Application.java | Bootstrap Spring Boot application for the new scenario |
| test/plugin/scenarios/lettuce-6.5.x-scenario/src/main/java/.../LettuceController.java | Implement sync/async Redis operations for tracing |
| apm-sniffer/apm-sdk-plugin/pom.xml | Replace lettuce-5.x-plugin with a new lettuce-plugins aggregator module |
| apm-sniffer/apm-sdk-plugin/lettuce-plugins/pom.xml | Declare lettuce-common and lettuce-6.5.x-plugin modules |
| apm-sniffer/apm-sdk-plugin/lettuce-plugins/lettuce-common/src/main/java/.../RedisChannelWriterInterceptor.java | Refactor and extract common instrumentation logic |
| apm-sniffer/apm-sdk-plugin/lettuce-plugins/lettuce-6.5.x-plugin/src/main/java/.../RedisChannelWriterInterceptorV6.java | Override command‐name extraction for Lettuce 6.5+ |
| CHANGES.md | Document release note for Lettuce 6.5+ support |
| .github/workflows/plugins-test.1.yaml | Enable CI job for the new lettuce-6.5.x-scenario |
apm-sniffer/apm-sdk-plugin/lettuce-plugins/lettuce-6.5.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/lettuce/v65/RedisChannelWriterInterceptorV6.java:26
protected String getCommandName(final ProtocolKeyword protocol) {
Sorry, something went wrong.
|
Your change somehow impacts the existing tests.
Could you recheck? |
Sorry, something went wrong.
|
Could you give me more explanations here about what is difference between v5 and v6.5?, besides the different witness class. |
Sorry, something went wrong.
|
Most are good. Please fix some nits and we should be able to merge this. |
Sorry, something went wrong.
|
Hi, i fixed the PR suggestions . |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.