Sentinel icon indicating copy to clipboard operation
Sentinel copied to clipboard

Fix [3462] Incorrect method of reading indicator files

Open manojks1999 opened this issue 1 year ago • 2 comments

Describe what this PR does / why we need it

Fixes for #3462

Does this pull request fix one issue?

Fixes #3462

Describe how you did it

Removed line 209 of sentinel-core/src/main/java/com/alibaba/csp/sentinel/node/metric/MetricSearcher.java

Describe how to verify it

Special notes for reviews

manojks1999 avatar Oct 27 '24 17:10 manojks1999

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Oct 27 '24 17:10 CLAassistant

This code is currently processing an index file. I'm having trouble mapping the issue description to the observed behavior - could you please elaborate on the exact conditions/reproduction steps you're encountering? Additionally, if we remove this line, the while condition would incorrectly compare Index with beginSecond, which is semantically invalid as these represent different dimensional units in our processing pipeline.

LearningGp avatar Sep 15 '25 12:09 LearningGp