crabapples
/
Sentinel
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

perf: improved Date formatter (#3466) 

* Improved Date formatter

* change in naming conv

* change in datetime format

* Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This commit is contained in:
manojks1999 2025-09-16 07:12:41 +05:30 committed by GitHub
parent d9398b4f75
commit 4a419818af
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 14 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
sentinel-core/src/main/java/com/alibaba/csp/sentinel/node/metric/MetricNode.java
View File

@ -15,9 +15,10 @@
*/ */
package com.alibaba.csp.sentinel.node.metric; package com.alibaba.csp.sentinel.node.metric;
import java.text.DateFormat; import java.time.Instant;
import java.text.SimpleDateFormat; import java.time.LocalDateTime;
import java.util.Date; import java.time.ZoneId;
import java.time.format.DateTimeFormatter;
/** /**
* Metrics data for a specific resource at given {@code timestamp}. * Metrics data for a specific resource at given {@code timestamp}.
@ -210,12 +211,19 @@ public class MetricNode {
* *
* @return string format of this. * @return string format of this.
*/ */
private static final DateTimeFormatter df = DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm:ss");
public String toFatString() { public String toFatString() {
DateFormat df = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
StringBuilder sb = new StringBuilder(32); StringBuilder sb = new StringBuilder(32);
sb.delete(0, sb.length()); sb.delete(0, sb.length());
sb.append(getTimestamp()).append("|");
sb.append(df.format(new Date(getTimestamp()))).append("|"); long timestamp = getTimestamp();
sb.append(timestamp).append("|");
LocalDateTime dateTime = LocalDateTime.ofInstant(Instant.ofEpochMilli(timestamp), ZoneId.systemDefault());
sb.append(df.format(dateTime)).append("|");
String legalName = getResource().replaceAll("\\|", "_"); String legalName = getResource().replaceAll("\\|", "_");
sb.append(legalName).append("|"); sb.append(legalName).append("|");
sb.append(getPassQps()).append("|"); sb.append(getPassQps()).append("|");