[GitHub] [skywalking] wu-sheng commented on pull request #5886: fix gateway plugin async finish repeatedly when fallback configured

2021-07-17 Thread GitBox
wu-sheng commented on pull request #5886: URL: https://github.com/apache/skywalking/pull/5886#issuecomment-881904451 > you mean just put the fallback behavior in the span of origin request rather than in a separate span? > I think that's might be better for most cases , since most of

[GitHub] [skywalking] wu-sheng commented on pull request #5886: fix gateway plugin async finish repeatedly when fallback configured

2021-07-17 Thread GitBox
wu-sheng commented on pull request #5886: URL: https://github.com/apache/skywalking/pull/5886#issuecomment-881891163 > The problem of the old code is that get the span from attributes within the execution of Mono would get span for latter request twice. so just move this part of code out

[GitHub] [skywalking] wu-sheng commented on pull request #5886: fix gateway plugin async finish repeatedly when fallback configured

2021-07-16 Thread GitBox
wu-sheng commented on pull request #5886: URL: https://github.com/apache/skywalking/pull/5886#issuecomment-881221354 Webflux case fails. -- 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

[GitHub] [skywalking] wu-sheng commented on pull request #5886: fix gateway plugin async finish repeatedly when fallback configured

2021-07-15 Thread GitBox
wu-sheng commented on pull request #5886: URL: https://github.com/apache/skywalking/pull/5886#issuecomment-881193468 ![image](https://user-images.githubusercontent.com/5441976/125898057-46471dca-b3c5-4522-b009-b58c0c079fba.png) Still fail. @yujiaxinlong -- This is an automated

[GitHub] [skywalking] wu-sheng commented on pull request #5886: fix gateway plugin async finish repeatedly when fallback configured

2021-07-13 Thread GitBox
wu-sheng commented on pull request #5886: URL: https://github.com/apache/skywalking/pull/5886#issuecomment-879051897 @yujiaxinlong Could you take a look? -- 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

[GitHub] [skywalking] wu-sheng commented on pull request #5886: fix gateway plugin async finish repeatedly when fallback configured

2021-07-13 Thread GitBox
wu-sheng commented on pull request #5886: URL: https://github.com/apache/skywalking/pull/5886#issuecomment-878952898 > @wu-sheng > kind of forgot about this PR, this issue seems still exist in latest version, is there anything I can do or just waiting for review of > @ascrutae

[GitHub] [skywalking] wu-sheng commented on pull request #5886: fix gateway plugin async finish repeatedly when fallback configured

2021-05-12 Thread GitBox
wu-sheng commented on pull request #5886: URL: https://github.com/apache/skywalking/pull/5886#issuecomment-839791341 Closing as no update. -- 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

[GitHub] [skywalking] wu-sheng commented on pull request #5886: fix gateway plugin async finish repeatedly when fallback configured

2020-11-23 Thread GitBox
wu-sheng commented on pull request #5886: URL: https://github.com/apache/skywalking/pull/5886#issuecomment-732563841 Please update the changelog. This is an automated message from the Apache Git Service. To respond to the