Skip to content

Commit f32c444

Browse files
committed
Minor update
Signed-off-by: Chojan Shang <[email protected]>
1 parent b7dbe5e commit f32c444

File tree

1 file changed

+7
-13
lines changed

1 file changed

+7
-13
lines changed

datafusion/functions/src/string/overlay.rs

+7-13
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ impl ScalarUDFImpl for OverlayFunc {
9191

9292
macro_rules! process_overlay {
9393
// For the three-argument case
94-
($string_array:expr, $characters_array:expr, $pos_num:expr, $is_view:expr) => {{
94+
($string_array:expr, $characters_array:expr, $pos_num:expr) => {{
9595
$string_array
9696
.iter()
9797
.zip($characters_array.iter())
@@ -125,7 +125,7 @@ macro_rules! process_overlay {
125125
}};
126126

127127
// For the four-argument case
128-
($string_array:expr, $characters_array:expr, $pos_num:expr, $len_num:expr, $is_view:expr) => {{
128+
($string_array:expr, $characters_array:expr, $pos_num:expr, $len_num:expr) => {{
129129
$string_array
130130
.iter()
131131
.zip($characters_array.iter())
@@ -180,8 +180,7 @@ pub fn string_overlay<T: OffsetSizeTrait>(args: &[ArrayRef]) -> Result<ArrayRef>
180180
let characters_array = as_generic_string_array::<T>(&args[1])?;
181181
let pos_num = as_int64_array(&args[2])?;
182182

183-
let result =
184-
process_overlay!(string_array, characters_array, pos_num, false)?;
183+
let result = process_overlay!(string_array, characters_array, pos_num)?;
185184
Ok(Arc::new(result) as ArrayRef)
186185
}
187186
4 => {
@@ -190,13 +189,8 @@ pub fn string_overlay<T: OffsetSizeTrait>(args: &[ArrayRef]) -> Result<ArrayRef>
190189
let pos_num = as_int64_array(&args[2])?;
191190
let len_num = as_int64_array(&args[3])?;
192191

193-
let result = process_overlay!(
194-
string_array,
195-
characters_array,
196-
pos_num,
197-
len_num,
198-
false
199-
)?;
192+
let result =
193+
process_overlay!(string_array, characters_array, pos_num, len_num)?;
200194
Ok(Arc::new(result) as ArrayRef)
201195
}
202196
other => {
@@ -212,7 +206,7 @@ pub fn string_view_overlay<T: OffsetSizeTrait>(args: &[ArrayRef]) -> Result<Arra
212206
let characters_array = as_string_view_array(&args[1])?;
213207
let pos_num = as_int64_array(&args[2])?;
214208

215-
let result = process_overlay!(string_array, characters_array, pos_num, true)?;
209+
let result = process_overlay!(string_array, characters_array, pos_num)?;
216210
Ok(Arc::new(result) as ArrayRef)
217211
}
218212
4 => {
@@ -222,7 +216,7 @@ pub fn string_view_overlay<T: OffsetSizeTrait>(args: &[ArrayRef]) -> Result<Arra
222216
let len_num = as_int64_array(&args[3])?;
223217

224218
let result =
225-
process_overlay!(string_array, characters_array, pos_num, len_num, true)?;
219+
process_overlay!(string_array, characters_array, pos_num, len_num)?;
226220
Ok(Arc::new(result) as ArrayRef)
227221
}
228222
other => {

0 commit comments

Comments
 (0)