Sentinel icon indicating copy to clipboard operation
Sentinel copied to clipboard

修复bug: NodeSelectorSlot只会记录初次调用记录

Open lookingatstarts opened this issue 10 months ago • 1 comments

Describe what this PR does / why we need it

NodeSelectorSlot只会在创建Node时候记录资源的调用关系,这样就只会记录初次调用关系。 eg : A B C三个资源 ContextUtil.enter("invoice-tree"); // 第一次调用 Entry a = SphU.entry("A"); Entry b = SphU.entry("B"); Entry c = SphU.entry("C"); c.exit(); b.exit(); a.exit(); // 第二次调用 a = SphU.entry("A"); Entry d = SphU.entry("D"); d.exit(); a.exit(); 仅仅会记录 A -> B -> C 调用关系,, 不会记录 A->D调用关系

Does this pull request fix one issue?

Describe how you did it

将NodeSelectorSlot记录调用的代码移出外围执行

Describe how to verify it

CtEntryTest有相关验证代码

Special notes for reviews

lookingatstarts avatar Feb 13 '25 06:02 lookingatstarts

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Feb 13 '25 06:02 CLAassistant

Please sign the CLA as per the instructions from CLAassistant.

LearningGp avatar Oct 09 '25 02:10 LearningGp

Due to this PR's inactivity and the unsigned CLA, we are temporarily closing it. Feel free to create a new PR if you still intend to contribute after addressing the CLA requirement.

LearningGp avatar Oct 16 '25 01:10 LearningGp