Skip to content

Comments

Added support for Lettuce reactive Redis commands#788

Open
wuwen5 wants to merge 8 commits intoapache:mainfrom
wuwen5:lettuce-reactive
Open

Added support for Lettuce reactive Redis commands#788
wuwen5 wants to merge 8 commits intoapache:mainfrom
wuwen5:lettuce-reactive

Conversation

@wuwen5
Copy link
Contributor

@wuwen5 wuwen5 commented Feb 8, 2026

  • If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
  • Update the CHANGES log.

Resolves apache/skywalking#12817

@wuwen5 wuwen5 requested a review from wu-sheng February 9, 2026 11:45
@wu-sheng
Copy link
Member

I am not sure whethere there is some issues in e2e. Many fails, but seem irrelevant.

@wu-sheng
Copy link
Member

I fixed the test framework via recent release PR(#791). Although two rocketmq cases have to been removed as it can't pass in CI in anyway, can't find out a clue.

Once I get that PR merged, I could update here, and your case should be able to run again.

@wu-sheng
Copy link
Member

Resolve conflicts as I put 9.7 out, and have fixed all things.

@wu-sheng
Copy link
Member

supported-framework doc should be updated as well, and supported versions should be in changes as well.

@wu-sheng
Copy link
Member

A key question is as following, we should consider this case. I think user could use like this, right?
Reactor-only context source: The snapshot is extracted via SKYWALKING_CONTEXT_SNAPSHOT from Reactor Context, which is only set by the spring-webflux plugin. If someone uses
Lettuce reactive without Spring WebFlux (e.g., standalone Reactor app), no context propagation occurs.

…main/java/org/apache/skywalking/apm/plugin/lettuce/common/RedisCommandEnhanceInfo.java

Co-authored-by: 吴晟 Wu Sheng <wu.sheng@foxmail.com>
@wuwen5
Copy link
Contributor Author

wuwen5 commented Feb 21, 2026

supported-framework doc should be updated as well, and supported versions should be in changes as well.

This PR does not change the supported version ranges of any frameworks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] webflux trace is seperated,there are mutilple traces but actual is only one trace

2 participants