refactor(query): pass timezone by reference to avoid Arc churn (#18998)

- Refactor the timezone plumbing so DateConverter, timestamp arithmetic helpers, JSON/date formatting, and all scalar/window consumers accept &TimeZone; cloning happens only when a jiff API needs ownership. This removes dozens of
    redundant Arc bumps and keeps behavior identical.
  - Update every downstream caller (functions, formats, window transforms, system/cloud history tables) to pass references, fixing the prior mismatches and addressing the jiff range clamp sites.

Author: TCeason

src/query/expression/src/types/date.rs

@@ -159,6 +159,6 @@ pub fn string_to_date(
 
 #[inline]
 pub fn date_to_string(date: impl AsPrimitive<i64>, tz: &TimeZone) -> impl Display {
-    let res = date.as_().to_date(tz.clone());
+    let res = date.as_().to_date(tz);
     strtime::format(DATE_FORMAT, res).unwrap()
 }

src/query/expression/src/types/timestamp.rs

@@ -192,6 +192,6 @@ pub fn string_to_timestamp(
 
 #[inline]
 pub fn timestamp_to_string(ts: i64, tz: &TimeZone) -> impl Display {
-    let zdt = ts.to_timestamp(tz.clone());
+    let zdt = ts.to_timestamp(tz);
     strtime::format(TIMESTAMP_FORMAT, &zdt).unwrap()
 }