[ISSUE#8481]Unified TRACE capacity building #8805
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
� Conflicts: � naming/src/main/java/com/alibaba/nacos/naming/push/v2/task/PushExecuteTask.java
KomachiSion
requested changes
Jul 26, 2022
|
|
||
| /** | ||
| * Trace event. | ||
| * | ||
| * @author yanda | ||
| */ | ||
| public class TraceEvent extends Event { | ||
| private TraceEventType type; |
There was a problem hiding this comment.
I think the type is String will be better.
| @@ -103,7 +103,6 @@ public boolean publish(Event event) { | |||
| boolean success = this.queue.offer(event); | |||
| if (!success) { | |||
| LOGGER.warn("Unable to plug in due to interruption, synchronize sending time, event : {}", event); | |||
| handleEvent(event); | |||
There was a problem hiding this comment.
remove call handleEvent logic, the warning log should be change also.
Codecov Report
@@ Coverage Diff @@
## develop #8805 +/- ##
=============================================
+ Coverage 43.07% 43.16% +0.09%
- Complexity 4693 4712 +19
=============================================
Files 904 912 +8
Lines 32479 32567 +88
Branches 3748 3757 +9
=============================================
+ Hits 13990 14059 +69
- Misses 17137 17152 +15
- Partials 1352 1356 +4
Continue to review full report at Codecov.
|
1. Event type add to traceEvent. 2. The event about instance add instancePort. 3. Add health check type to healthChangeEvent. 4. Other fixup.
KomachiSion
approved these changes
Jul 29, 2022
li-xiao-shuang
pushed a commit
that referenced
this pull request
Aug 31, 2022
* Add exception detail stacks when connect to server failed. * [ISSUE#8653]Fix health check bug (#8639) * fix health bug. * fix health bug. * fix health bug. * test update * format code * 注释 * check service exist * check service exist * test fail * user group() * add unit test. * add unit test. * add unit test. * add unit test. * add unit test. * add unit test. * Add comment for config aspect. * [ISSUE#8481]Unified TRACE capacity building (#8805) * [ISSUE#8481]Unified TRACE capacity building * [ISSUE#8481]Unified TRACE capacity building 1. Event type add to traceEvent. 2. The event about instance add instancePort. 3. Add health check type to healthChangeEvent. 4. Other fixup. * Add Generic Type for Simple Cache Map (#8812) * import config file auth fail (#8844) * import config file auth fail * run ci * run ci2 * Feat/calculate ip count when use batchregister (#8858) Feat/calculate ip count when use batchregister * [ISSUE #8880] add a request exception handler (#8881) * [ISSUE #8880] add a request exception handler Close #8880 * [ISSUE #8880] retry ci * Upgrade to 2.1.1 (#8899) * [ISSUE#8910] bug fix calcu ipCount (#8911) * bug fix calcu ipCount * update logic * judge endpoint (#8933) * [ISSUE #8925] fix that the value of hasQueryString is always false (#8927) Close #8925 * [ISSUES#8905]simplify some code (#8908) * refactor: 💡 update console-ui theme (#8951) ✅ Closes: #8950 * Update IoUtils.java (#8948) An IOException has been catch，so throws IOException Is invalid * [ISSUE #8931] Fix bug :BatchInstanceData cannot Serializable question (#8940) * [fix] fix BatchInstanceData cannot Serializable question * fix update serialVersionUID auto generate * [ISSUE #8925] fix that the replaceAll operation is invalid (#8929) Close #8925 * fix #8947. (#8959) * judge endpoint * add some junit test (#8966) * add some junit test * fix some junit * add license * optimize: internationalization of product description content on landing pages (#8956) * Improve code specifications. (#8965) Flip the call to equals method to avoid null pointer exception. * fix LookupFactoryTest ut / ci (#8985) * fix ci * run ci * check cla report * check style * add some auth plugin test (#8988) * add some auth plugin test * ru ci * remove namespaceShowName uniqueness check (#8989) * add unit test for ReflectUtils (#8999) * add unit test for ReflectUtils * reformat code * [ISSUE #8993]Close input stream and output stream by try with resource[nacos-common] (#8997) * [ISSUE #8993]Close input stream and output stream by try with resource[nacos-common] * [ISSUE #8993]Close input stream and output stream by try with resource[nacos-common] * [ISSUE #8993]Close input stream and output stream by try with resource[nacos-common] * add some utils UT (#9006) add some utils UT * Fix grpc push data not filter by cluster. (#9019) * [#344] rename initialization sql files (#8969) * [ISSUE #9014] performance optimization--refacte TpsMonitorPoint (#9021) * performance optimization * fix check style * fix test unit * fix test unit for `getTimeFormatOfSecond` method. * fix unit test for LimiterTest. * fix error usage case for asserEquals method；eliminate interactions between test methods * Fix spelling mistakes. (#9026) * [ISSUE #9013] enhance grpc client (#9017) * fix issue #9013 * reformat code style * reformat code style * reformat code style * reformat code style * reformat code style * reformat code style * reformat code style * change config name * do some enhance * add queue size to configProperties * reformat code style * Change configuration load order * Fix ServerListMgr is not shutdown. (#9048) * set server.error.include-message to ALWAYS (#9045) * [ISSUE #8934] add header back after convert request to jsonBytes (#8935) * [ISSUE #9023] fix corner case config dataId 'cipher-' that cannot create (#9024) * [ISSUE #9023] fix corner case config dataId 'cipher-' that cannot create * [ISSUE #9023] remove unused import * [ISSUE #9023] PMD fix * [ISSUE #9023] add comments * [ISSUE #9023] should return first token instead of any * [ISSUE #9023] add more EncryptionHandler ut * [ISSUE #9023] fix style * [ISSUE #9023] fix style, keep indentation in blank line Co-authored-by: KomachiSion <xiweng.yy@alibaba-inc.com> Co-authored-by: 胡俊 <510830970@qq.com> Co-authored-by: MajorHe1 <53824889+MajorHe1@users.noreply.github.com> Co-authored-by: GuiShao <zhupeng.zhu@alibaba-inc.com> Co-authored-by: ZhangShenao <15201440436@163.com> Co-authored-by: chenhao26 <35129699+chenhao26-nineteen@users.noreply.github.com> Co-authored-by: onewe <admin@onew.me> Co-authored-by: 胡俊 <hujun3@xiaomi.com> Co-authored-by: liushaoyu-soft <455028117@qq.com> Co-authored-by: heimanba <371510756@qq.com> Co-authored-by: 无处不在 <42382506+wuchubuzai2018@users.noreply.github.com> Co-authored-by: Weizhan Yun <yunweizhan@gmail.com> Co-authored-by: liaochuntao <liaochuntao@live.com> Co-authored-by: @xiaochangbai <704566072@qq.com> Co-authored-by: shiherua <56618514+shiherua@users.noreply.github.com> Co-authored-by: Karson <karsontao@hotmail.com> Co-authored-by: yjqg6666 <jinking.this@gmail.com> Co-authored-by: 阿魁 <670569467@qq.com> Co-authored-by: wuwen <wuwen.55@aliyun.com> Co-authored-by: hutt <200714916@qq.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[[ISSUE#8481]Unified TRACE capacity building]
For #8481