Skip to content

Commit

Permalink
chore(log): suppress unnecessary warning (#7676)
Browse files Browse the repository at this point in the history
Change this warning message into a debug message, because it's the expected behavior during backfill. Otherwise it can overwhelm the log stream during a large MV creation.

Approved-By: wenym1
  • Loading branch information
zwang28 authored Feb 3, 2023
1 parent a97e94f commit f1c4558
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/storage/src/hummock/iterator/merge_inner.rs
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ use std::future::Future;
use std::ops::{Deref, DerefMut};

use risingwave_hummock_sdk::key::{FullKey, TableKey, UserKey};
use tracing::warn;

use crate::hummock::iterator::{DirectionEnum, HummockIterator, HummockIteratorDirection};
use crate::hummock::value::HummockValue;
Expand Down Expand Up @@ -261,7 +260,9 @@ impl<'a, T: Ord> Drop for PeekMutGuard<'a, T> {
/// call `PeekMut::pop` on the `PeekMut` and recycle the node to the unused list.
fn drop(&mut self) {
if let Some(peek) = self.peek.take() {
warn!("PeekMut are dropped without used. May be caused by future cancellation");
tracing::debug!(
"PeekMut are dropped without used. May be caused by future cancellation"
);
let top = PeekMut::pop(peek);
self.unused.push_back(top);
}
Expand Down

0 comments on commit f1c4558

Please sign in to comment.