Re: [PR] Fix method name missing in spring-plugins:scheduled-annotation-plugin with spring 6.1.x [skywalking-java]

2024-05-14 Thread via GitHub


wu-sheng merged PR #691:
URL: https://github.com/apache/skywalking-java/pull/691


-- 
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-unsubscr...@skywalking.apache.org

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



Re: [PR] Fix method name missing in spring-plugins:scheduled-annotation-plugin with spring 6.1.x [skywalking-java]

2024-05-14 Thread via GitHub


wu-sheng commented on PR #691:
URL: https://github.com/apache/skywalking-java/pull/691#issuecomment-2109895082

   I can see RocketMQ fails again. From what I see, 50% chance 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 specific comment.

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

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



Re: [PR] Fix method name missing in spring-plugins:scheduled-annotation-plugin with spring 6.1.x [skywalking-java]

2024-05-14 Thread via GitHub


CzyerChen commented on PR #691:
URL: https://github.com/apache/skywalking-java/pull/691#issuecomment-2109729418

   > Are we sending too frequently? The test GHA env is only 3u8g for all 
runtime. It could be super slow.
   
   May have an impact.


-- 
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-unsubscr...@skywalking.apache.org

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



Re: [PR] Fix method name missing in spring-plugins:scheduled-annotation-plugin with spring 6.1.x [skywalking-java]

2024-05-14 Thread via GitHub


wu-sheng commented on PR #691:
URL: https://github.com/apache/skywalking-java/pull/691#issuecomment-2109656114

   Are we sending too frequently? The test GHA env is only 3u8g for all 
runtime. It could be super slow.


-- 
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-unsubscr...@skywalking.apache.org

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



Re: [PR] Fix method name missing in spring-plugins:scheduled-annotation-plugin with spring 6.1.x [skywalking-java]

2024-05-14 Thread via GitHub


CzyerChen commented on PR #691:
URL: https://github.com/apache/skywalking-java/pull/691#issuecomment-2109653475

   > I can see RocketMQ case fails again.
   
   The problem is not reproduced locally, and the sending and receiving of 
async messages are enhanced.
   
   Please do inform me when the recurrence of the problem.


-- 
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-unsubscr...@skywalking.apache.org

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



Re: [PR] Fix method name missing in spring-plugins:scheduled-annotation-plugin with spring 6.1.x [skywalking-java]

2024-05-13 Thread via GitHub


CzyerChen commented on PR #691:
URL: https://github.com/apache/skywalking-java/pull/691#issuecomment-2109206083

   > I can see RocketMQ case fails again.
   
   OK,I will check RocketMQ case first.


-- 
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-unsubscr...@skywalking.apache.org

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



Re: [PR] Fix method name missing in spring-plugins:scheduled-annotation-plugin with spring 6.1.x [skywalking-java]

2024-05-13 Thread via GitHub


wu-sheng commented on PR #691:
URL: https://github.com/apache/skywalking-java/pull/691#issuecomment-2108028821

   I can see RocketMQ case fails 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.

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

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



Re: [PR] Fix method name missing in spring-plugins:scheduled-annotation-plugin with spring 6.1.x [skywalking-java]

2024-05-13 Thread via GitHub


wu-sheng commented on code in PR #691:
URL: https://github.com/apache/skywalking-java/pull/691#discussion_r1598620639


##
CHANGES.md:
##
@@ -11,6 +11,7 @@ Release Notes.
 * Use a daemon thread to flush logs.
 * Fix typos in `URLParser`.
 * Add support for `Derby`/`Sybase`/`SQLite`/`DB2`/`OceanBase` jdbc url format 
in `URLParser`.
+* Optimize spring-plugins:scheduled-annotation-plugin compatibility.

Review Comment:
   ```suggestion
   * Optimize spring-plugins:scheduled-annotation-plugin compatibility about 
Spring 6.1.x support.
   ```



-- 
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-unsubscr...@skywalking.apache.org

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