fix: some cr comments

src/mito2/src/compaction.rs

@@ -12,6 +12,14 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+mod buckets;
+mod picker;
+mod task;
+#[cfg(test)]
+mod test_util;
+mod twcs;
+mod window;
+
 use std::collections::HashMap;
 use std::sync::Arc;
 use std::time::{Duration, Instant};
@@ -54,14 +62,6 @@ use crate::sst::file_purger::FilePurgerRef;
 use crate::sst::version::LevelMeta;
 use crate::worker::WorkerListener;
 
-mod buckets;
-mod picker;
-mod task;
-#[cfg(test)]
-mod test_util;
-mod twcs;
-mod window;
-
 /// Region compaction request.
 pub struct CompactionRequest {
     pub(crate) engine_config: Arc<MitoConfig>,

src/mito2/src/compaction/window.rs

@@ -191,6 +191,7 @@ fn assign_files_to_time_windows<'a>(
         }
         let (start, end) = file.time_range();
         let bounds = file_time_bucket_span(
+            // safety: converting whatever timestamp to seconds will not overflow.
             start.convert_to(TimeUnit::Second).unwrap().value(),
             end.convert_to(TimeUnit::Second).unwrap().value(),
             bucket_sec,

src/session/src/table_name.rs

@@ -28,6 +28,7 @@ pub fn table_name_to_full_name(
     let obj_name = ParserContext::parse_table_name(name, query_ctx.sql_dialect())?;
 
     let (catalog, schema, table) = table_idents_to_full_name(&obj_name, query_ctx)?;
+    // todo(hl): also check if schema matches when rbac is ready. https://github.com/GreptimeTeam/greptimedb/pull/3988/files#r1608687652
     ensure!(
         catalog == query_ctx.current_catalog(),
         PermissionDeniedSnafu {