Re: [PR] Fix method name missing in spring-plugins:scheduled-annotation-plugin with spring 6.1.x [skywalking-java]
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]
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]
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]
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]
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]
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]
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]
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