Skip to content

Commit a17eb60

Browse files
author
hexudong
committed
code style and CHANGES.md fixed
1 parent 5e9e53f commit a17eb60

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

CHANGES.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ Release Notes.
1414
* Bump up cli to the 0.15.0-dev.latest(77b4c49e89c9c000278f44e62729d534f2ec842e) in e2e.
1515
* Bump up apache parent pom to v35.
1616
* Update Maven to 3.6.3 in mvnw.
17-
* Fix oom due to too many span log.
17+
* Fix OOM due to too many span logs.
1818

1919
All issues and pull requests are [here](https://github.com/apache/skywalking/milestone/242?closed=1)
2020

apm-sniffer/apm-agent-core/src/main/java/org/apache/skywalking/apm/agent/core/context/trace/AbstractTracingSpan.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ public AbstractTracingSpan log(Throwable t) {
175175
if (!errorOccurred && ServiceManager.INSTANCE.findService(StatusCheckService.class).isError(t)) {
176176
errorOccurred();
177177
}
178-
if(logs.size() >= Config.Agent.LOG_LIMIT_PER_SPAN){
178+
if (logs.size() >= Config.Agent.LOG_LIMIT_PER_SPAN) {
179179
return this;
180180
}
181181
logs.add(new LogDataEntity.Builder().add(new KeyValuePair("event", "error"))
@@ -199,7 +199,7 @@ public AbstractTracingSpan log(long timestampMicroseconds, Map<String, ?> fields
199199
if (logs == null) {
200200
logs = new LinkedList<>();
201201
}
202-
if(logs.size() >= Config.Agent.LOG_LIMIT_PER_SPAN){
202+
if (logs.size() >= Config.Agent.LOG_LIMIT_PER_SPAN) {
203203
return this;
204204
}
205205
LogDataEntity.Builder builder = new LogDataEntity.Builder();

0 commit comments

Comments
 (0)