Apply suggestions from code review

GreptimeTeam · Nov 17, 2023 · fb23173 · fb23173
1 parent f99657c
commit fb23173
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/query/src/range_select/plan_rewrite.rs b/src/query/src/range_select/plan_rewrite.rs
@@ -343,7 +343,8 @@ impl RangePlanRewriter {
                         .map(|key| Expr::Column(Column::new(Some(table_ref.clone()), key)))
                         .collect();
                     // If the user does not specify a primary key when creating a table,
-                    // then by default all data will be aggregated into one time series, which is equivalent to using `by(1)` in SQL
+                    // then by default all data will be aggregated into one time series,
+                    // which is equivalent to using `by(1)` in SQL
                     if default_by.is_empty() {
                         default_by = vec![Expr::Literal(ScalarValue::Int64(Some(1)))];
                     }

diff --git a/tests/cases/standalone/common/range/by.sql b/tests/cases/standalone/common/range/by.sql
@@ -57,4 +57,4 @@ INSERT INTO TABLE host VALUES
 
 SELECT ts, max(val) RANGE '5s' FROM host ALIGN '20s' ORDER BY ts;
 
-DROP TABLE host;
+DROP TABLE host;
Original file line number	Diff line number	Diff line change
Expand Up		@@ -57,4 +57,4 @@ INSERT INTO TABLE host VALUES

		SELECT ts, max(val) RANGE '5s' FROM host ALIGN '20s' ORDER BY ts;

		DROP TABLE host;
		DROP TABLE host;