kazusa
/
skywalking
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Make the duration error message more specific (#5755)

This commit is contained in:
Hoshea Jiang 2020-10-30 22:55:03 +08:00 committed by GitHub
parent 8a2a0008ba
commit 0fb23b87f8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/query/DurationUtils.java
View File

@ -30,6 +30,8 @@ import org.joda.time.format.DateTimeFormatter;
public enum DurationUtils {
INSTANCE;
private static final int MAX_TIME_SPAN = 500;
private static final DateTimeFormatter YYYY_MM_DD = DateTimeFormat.forPattern("yyyy-MM-dd");
private static final DateTimeFormatter YYYY_MM_DD_HH = DateTimeFormat.forPattern("yyyy-MM-dd HH");
private static final DateTimeFormatter YYYY_MM_DD_HHMM = DateTimeFormat.forPattern("yyyy-MM-dd HHmm");
@ -119,9 +121,11 @@ public enum DurationUtils {
break;
}
i++;
if (i > 500) {
if (i > MAX_TIME_SPAN) {
throw new UnexpectedException(
"Duration data error, step: " + step.name() + ", start: " + startTimeBucket + ", end: " + endTimeBucket);
"Duration data error, the span between the start time and the end time is too large, step: "
+ step.name() + ", start: " + startTimeBucket + ", end: " + endTimeBucket
+ ", max time span: " + MAX_TIME_SPAN);
}
}
while (endTimeBucket != durations.get(durations.size() - 1).getPoint());