Skip to content

Commit 9a11088

Browse files
committed
address comment
1 parent 5803583 commit 9a11088

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

datafusion/functions-table/src/generate_series.rs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,20 +30,23 @@ use parking_lot::RwLock;
3030
use std::fmt;
3131
use std::sync::Arc;
3232

33+
/// Indicates the arguments used for generating a series.
3334
#[derive(Debug, Clone)]
3435
enum GenSeriesArgs {
36+
/// ContainsNull signifies that at least one argument(start, end, step) was null, thus no series will be generated.
3537
ContainsNull,
38+
/// AllNotNullArgs holds the start, end, and step values for generating the series when all arguments are not null.
3639
AllNotNullArgs { start: i64, end: i64, step: i64 },
3740
}
3841

39-
/// Table that generates a series of integers from `start`(inclusive) to `end`(inclusive)
42+
/// Table that generates a series of integers from `start`(inclusive) to `end`(inclusive), incrementing by step
4043
#[derive(Debug, Clone)]
4144
struct GenerateSeriesTable {
4245
schema: SchemaRef,
4346
args: GenSeriesArgs,
4447
}
4548

46-
/// Table state that generates a series of integers from `start`(inclusive) to `end`(inclusive)
49+
/// Table state that generates a series of integers from `start`(inclusive) to `end`(inclusive), incrementing by step
4750
#[derive(Debug, Clone)]
4851
struct GenerateSeriesState {
4952
schema: SchemaRef,

0 commit comments

Comments
 (0)