fix: tests

Author: Weijun-H

datafusion/functions/src/datetime/now.rs

@@ -15,7 +15,6 @@
 // specific language governing permissions and limitations
 // under the License.
 
-use arrow::array::timezone;
 use arrow::datatypes::DataType::Timestamp;
 use arrow::datatypes::TimeUnit::Nanosecond;
 use arrow::datatypes::{DataType, Field, FieldRef};
@@ -81,7 +80,7 @@ impl ScalarUDFImpl for NowFunc {
     fn return_field_from_args(&self, _args: ReturnFieldArgs) -> Result<FieldRef> {
         Ok(Field::new(
             self.name(),
-            Timestamp(Nanosecond, Some("+00:00".into())),
+            Timestamp(Nanosecond, Some("+00".into())),
             false,
         )
         .into())
@@ -113,7 +112,7 @@ impl ScalarUDFImpl for NowFunc {
             .config_options
             .as_ref()
             .map(|opts| opts.execution.time_zone.as_str())
-            .unwrap_or("+00:00");
+            .unwrap_or("+00");
 
         Ok(ExprSimplifyResult::Simplified(Expr::Literal(
             ScalarValue::TimestampNanosecond(now_ts, Some(timezone.into())),

datafusion/sqllogictest/test_files/timestamps.slt

@@ -73,6 +73,16 @@ true
 ##########
 ## Current time Tests
 ##########
+statement ok
+SET TIME ZONE = '+08'
+
+query T
+select arrow_typeof(now());
+----
+Timestamp(Nanosecond, Some("+08"))
+
+statement ok
+SET TIME ZONE = '+00'
 
 query B
 select cast(now() as time) = current_time();