yswdqz commented on code in PR #9988:
URL: https://github.com/apache/skywalking/pull/9988#discussion_r1026564667
##########
oap-server/analyzer/log-analyzer/src/main/java/org/apache/skywalking/oap/log/analyzer/dsl/spec/extractor/ExtractorSpec.java:
##########
@@ -269,13 +274,12 @@ public void slowSql(@DelegatesTo(SlowSqlSpec.class) final
Closure<?> cl) {
DatabaseSlowStatementBuilder builder = new
DatabaseSlowStatementBuilder(namingControl);
builder.setLayer(Layer.nameOf(log.getLayer()));
- LocalDateTime localDateTime =
Instant.ofEpochSecond(log.getTimestamp()).atZone(ZoneId.systemDefault()).toLocalDateTime();
- String timeBucket = DTF.format(localDateTime);
- builder.setTimeBucket(Long.parseLong(timeBucket));
+ long timeBucket = TimeBucket.getTimeBucket(log.getTimestamp() * 1000,
DownSampling.Minute);
Review Comment:
No, but for more time user will use lua(os.time() to get current timestamp)
in fluent-bit. So I use it.
I also feel that this way maybe not good.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
