Skip to content

Commit

Permalink
[MINOR] Adjust log level for PushDataHandler#checkDiskFullAndSplit
Browse files Browse the repository at this point in the history
…from debug to trace

### What changes were proposed in this pull request?

As title

### Why are the changes needed?

This PR modifies the log level from debug to trace for the `PushDataHandler#checkDiskFullAndSplit` method, which is invoked with each `PushData` request. This change is aimed at addressing the issue of excessive log volume caused by the high frequency of `PushData` requests.

### Does this PR introduce _any_ user-facing change?

No

### How was this patch tested?

Pass GA

Closes apache#2007 from cfmcgrady/check-disk-full-and-split-log-level.

Authored-by: Fu Chen <[email protected]>
Signed-off-by: mingji <[email protected]>
  • Loading branch information
cfmcgrady authored and FMX committed Oct 20, 2023
1 parent f778324 commit 5b6e197
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1209,11 +1209,11 @@ class PushDataHandler extends BaseMessageHandler with Logging {
softSplit: AtomicBoolean,
callback: RpcResponseCallback): Boolean = {
val diskFull = checkDiskFull(fileWriter)
logDebug(
logTrace(
s"""
|CheckDiskFullAndSplit in
|diskFull:$diskFull,
|partitionSplitMinimumSize: $partitionSplitMinimumSize,
|partitionSplitMinimumSize:$partitionSplitMinimumSize,
|splitThreshold:${fileWriter.getSplitThreshold()},
|fileLength:${fileWriter.getFileInfo.getFileLength}
|fileName:${fileWriter.getFileInfo.getFilePath}
Expand All @@ -1225,7 +1225,7 @@ class PushDataHandler extends BaseMessageHandler with Logging {
softSplit.set(true)
} else {
callback.onSuccess(ByteBuffer.wrap(Array[Byte](StatusCode.HARD_SPLIT.getValue)))
logDebug(
logTrace(
s"""
|CheckDiskFullAndSplit hardSplit
|diskFull:$diskFull,
Expand Down

0 comments on commit 5b6e197

Please sign in to comment.