Skip to content

Commit

Permalink
[controller][server] Reduce logging in controller, server (#577)
Browse files Browse the repository at this point in the history
For stores without pushstatus system store, controller logs get flooded with full excception stacktrace. Also in server most of the time RMD file sst writer are empty, so do not print any warn log for it.
Co-authored-by: Sourav Maji <[email protected]>
  • Loading branch information
majisourav99 authored Aug 9, 2023
1 parent ceb2901 commit a73b77c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,7 @@ public Map<String, String> sync() {
+ recordNumInLastSSTFile + ", latency(ms): " + LatencyUtils.getElapsedTimeInMs(startMs));
}
}
} else {
} else if (!isRMD) {
LOGGER.warn(
"Sync gets invoked for store: {}, partition id: {}, but the last sst file: {} is empty",
storeName,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ private void scanDaVinciPushStatus() {
try {
pushStatus = future.get();
} catch (Exception e) {
LOGGER.error("Caught exception when getting future result of push status.", e);
LOGGER.error("Caught exception when getting future result of push status : " + e.getMessage());
continue;
}
ExecutionStatusWithDetails daVinciStatus = pushStatus.getDaVinciStatus();
Expand Down

0 comments on commit a73b77c

Please sign in to comment.