You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@skywalking.apache.org by GitBox <gi...@apache.org> on 2021/02/08 08:14:10 UTC

[GitHub] [skywalking] RDyang opened a new issue #6348: 集成Spring Webflux后链路断开

RDyang opened a new issue #6348:
URL: https://github.com/apache/skywalking/issues/6348


   Please answer these questions before submitting your issue.
   
   - Why do you submit this issue?
   - [X] Question or discussion
   - [ ] Bug
   - [ ] Requirement
   - [ ] Feature or performance improvement
   
   ___
   ### Question
   - What do you want to know?
   -  我把 ***webflux-scenario***中 projectA 的 spring-boot-starter-web依赖换成了spring-boot-starter-webflux后，请求/testcase接口会出现链路断开的情况。不知道各位大佬是否知道这个情况？
   这样导致我们集成以后几乎无法追踪调用链路。
   - 
   ![image](https://user-images.githubusercontent.com/20990746/107190622-fadc2e00-6a25-11eb-8dd0-7e2f91f9305f.png)
   ![image](https://user-images.githubusercontent.com/20990746/107190761-2e1ebd00-6a26-11eb-8b9a-8f8c74a3751c.png)
   
   
   
   ___
   ### Bug
   - Which version of SkyWalking, OS, and JRE?
    源码版本 skywalking-apm-8.3.0
   skywalking ui 版本：apache-skywalking-apm-es7-8.4.0
   OS:windows10
   JDK:JDK11.0.5
   - Which company or project?
    
   - What happened?
   If possible, provide a way to reproduce the error. e.g. demo application, component version.
   webflux-projectA-scenario pom.xml
    ```
   <dependencies>
           <dependency>
               <groupId>org.springframework.boot</groupId>
               <artifactId>spring-boot-starter-webflux</artifactId>
               <version>${test.framework.version}</version>
           </dependency>
           <dependency>
               <groupId>org.apache.httpcomponents</groupId>
               <artifactId>httpclient</artifactId>
               <version>4.5.6</version>
           </dependency>
   <!--        <dependency>-->
   <!--            <groupId>org.springframework</groupId>-->
   <!--            <artifactId>spring-webflux</artifactId>-->
   <!--            <version>5.2.9.RELEASE</version>-->
   <!--        </dependency>-->
   <!--        <dependency>-->
   <!--            <groupId>org.springframework.boot</groupId>-->
   <!--            <artifactId>spring-boot-starter-reactor-netty</artifactId>-->
   <!--            <version>2.3.4.RELEASE</version>-->
   <!--        </dependency>-->
       </dependencies>
   ```
   
    


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [skywalking] RDyang commented on issue #6348: 集成Spring Webflux后链路断开

Posted by GitBox <gi...@apache.org>.
RDyang commented on issue #6348:
URL: https://github.com/apache/skywalking/issues/6348#issuecomment-775048857


   I want to use Webflux and Netty, not SpringMVC and Tomcat.
   so I modified <url>https://github.com/apache/skywalking/blob/master/test/plugin/scenarios/webflux-scenario/webflux-projectA-scenario/pom.xml</url> because my project use *spring-boot-starter-webflux*, the result is just like the picture .there are two different spanId(maybe traceId?).
   I think it is wrong, the spanId of ***/testcase*** and ***/testcase/webclient/server*** should be the same
   ```
   <dependencies>
          <dependency>
              <groupId>org.springframework.boot</groupId>
              <artifactId>spring-boot-starter-webflux</artifactId>
              <version>${test.framework.version}</version>
          </dependency>
          <dependency>
              <groupId>org.apache.httpcomponents</groupId>
              <artifactId>httpclient</artifactId>
              <version>4.5.6</version>
          </dependency>
   <!--        <dependency>-->
   <!--            <groupId>org.springframework</groupId>-->
   <!--            <artifactId>spring-webflux</artifactId>-->
   <!--            <version>5.2.9.RELEASE</version>-->
   <!--        </dependency>-->
   <!--        <dependency>-->
   <!--            <groupId>org.springframework.boot</groupId>-->
   <!--            <artifactId>spring-boot-starter-reactor-netty</artifactId>-->
   <!--            <version>2.3.4.RELEASE</version>-->
   <!--        </dependency>-->
      </dependencies>
   ```
   ![image](https://user-images.githubusercontent.com/20990746/107205281-abebc400-6a38-11eb-82ef-2ab2a0e9d9e6.png)
   ![image](https://user-images.githubusercontent.com/20990746/107205335-b8701c80-6a38-11eb-8134-ea8e54e316aa.png)
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [skywalking] RDyang commented on issue #6348: 集成Spring Webflux后链路断开

Posted by GitBox <gi...@apache.org>.
RDyang commented on issue #6348:
URL: https://github.com/apache/skywalking/issues/6348#issuecomment-775081493


   I think test case should be coverd all 2 projects both use Spring Webflux and Netty, not one Netty , another Tomcat.
    Thank you very much for your reply . 
   I will study how to run tests and verify my case. Thanks again.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [skywalking] RDyang commented on issue #6348: 集成Spring Webflux后链路断开

Posted by GitBox <gi...@apache.org>.
RDyang commented on issue #6348:
URL: https://github.com/apache/skywalking/issues/6348#issuecomment-775005164


   ok，when I use *spring-boot-starter-webflux* instead of *spring-boot-starter-web* in project *webflux-projectA-scenario* then call "/projectA/testcase" , there are two spanId created.
   And I'm sorry I am not clear the reason of issue closed, only because of using Chinese or it would be fixed on 8.5.0?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [skywalking] wu-sheng closed issue #6348: 集成Spring Webflux后链路断开

Posted by GitBox <gi...@apache.org>.
wu-sheng closed issue #6348:
URL: https://github.com/apache/skywalking/issues/6348


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [skywalking] wu-sheng commented on issue #6348: 集成Spring Webflux后链路断开

Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #6348:
URL: https://github.com/apache/skywalking/issues/6348#issuecomment-775007300


   Because of Chinese. We need to keep all discussion available for global. Webflux tested by this, https://github.com/apache/skywalking/tree/master/test/plugin/scenarios/webflux-scenario. 
   And if you are using SpringMVC, you don't need this plugin.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [skywalking] wu-sheng commented on issue #6348: 集成Spring Webflux后链路断开

Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #6348:
URL: https://github.com/apache/skywalking/issues/6348#issuecomment-775052420


   We can't debug your codes. Because the work loads are crazy. Learn how to debug and how plugin works, by following the plugin development document.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [skywalking] wu-sheng commented on issue #6348: 集成Spring Webflux后链路断开

Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #6348:
URL: https://github.com/apache/skywalking/issues/6348#issuecomment-774959440


   Please use English only on the GitHub.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [skywalking] q82184828 commented on issue #6348: 集成Spring Webflux后链路断开

Posted by GitBox <gi...@apache.org>.
q82184828 commented on issue #6348:
URL: https://github.com/apache/skywalking/issues/6348#issuecomment-920546537


   i alse met this problem, have you solve it ?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [skywalking] RDyang commented on issue #6348: 集成Spring Webflux后链路断开

Posted by GitBox <gi...@apache.org>.
RDyang commented on issue #6348:
URL: https://github.com/apache/skywalking/issues/6348#issuecomment-775049288


   The right result should like this:
   ![image](https://user-images.githubusercontent.com/20990746/107208913-2a4a6500-6a3d-11eb-85cc-d0f4d244c78d.png)
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [skywalking] wu-sheng commented on issue #6348: 集成Spring Webflux后链路断开

Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #6348:
URL: https://github.com/apache/skywalking/issues/6348#issuecomment-775052747


   Our tests are typical, if you want to enhance for more cases, we are welcome. 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [skywalking] wu-sheng commented on issue #6348: 集成Spring Webflux后链路断开

Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #6348:
URL: https://github.com/apache/skywalking/issues/6348#issuecomment-775087397


   In the open source, there is nothing called should as demanding. If you want to enhance, welcome. But we can't do everything.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org