docs: move avro upgrade note to 54 guide

Author: getChan

docs/source/library-user-guide/upgrading/53.0.0.md

@@ -472,42 +472,3 @@ Now:
 +-------+
 0 row(s) fetched.
 ```
-
-### Avro API, feature, and timestamp decoding changes
-
-As part of the Avro reader migration (see [#17861]), DataFusion now delegates
-Avro-to-Arrow type decoding to `arrow-avro`, and several Avro-related
-APIs/feature wiring changed:
-
-- `DataFusionError::AvroError` has been removed.
-- `From<apache_avro::Error> for DataFusionError` has been removed.
-- Avro crate re-export changed:
-  - Before: `datafusion::apache_avro`
-  - After: `datafusion::arrow_avro`
-- Cargo feature wiring changed:
-  - `datafusion` crate `avro` feature no longer enables `datafusion-common/avro`
-  - `datafusion-proto` crate `avro` feature no longer enables `datafusion-common/avro`
-- **Avro timestamp logical type interpretation changed.** Notable effects:
-  - Avro `timestamp-*` logical types are read as UTC timezone-aware Arrow timestamps
-    (`Timestamp(..., Some("+00:00"))`)
-  - Avro `local-timestamp-*` logical types remain timezone-naive
-    (`Timestamp(..., None)`)
-
-**Who is affected:**
-
-- Users matching on `DataFusionError::AvroError`
-- Users importing `datafusion::apache_avro`
-- Users depending on the old `datafusion-common/avro` feature wiring
-- Users relying on previous Avro timestamp logical type behavior
-
-**Migration guide:**
-
-- Replace `datafusion::apache_avro` imports with `datafusion::arrow_avro`.
-- Update error handling code that matches on `DataFusionError::AvroError` to use
-  the current error surface.
-- If you depend on Avro feature propagation, update Cargo feature expectations:
-  `datafusion`/`datafusion-proto` `avro` no longer enables `datafusion-common/avro`.
-- Validate timestamp handling where timezone semantics matter:
-  `timestamp-*` is UTC timezone-aware, while `local-timestamp-*` is timezone-naive.
-
-[#17861]: https://github.com/apache/datafusion/pull/17861

docs/source/library-user-guide/upgrading/54.0.0.md

@@ -77,7 +77,7 @@ fn apply_expressions(
 
 **Node whose only expressions are in `output_ordering()` (e.g. a synthetic test node with no owned expression fields):**
 
-````rust,ignore
+```rust,ignore
 fn apply_expressions(
     &self,
     f: &mut dyn FnMut(&dyn PhysicalExpr) -> Result<TreeNodeRecursion>,
@@ -90,6 +90,7 @@ fn apply_expressions(
     }
     Ok(tnr)
 }
+```
 
 ### `ExecutionPlan::partition_statistics` now returns `Arc<Statistics>`
 
@@ -101,7 +102,7 @@ fn apply_expressions(
 fn partition_statistics(&self, partition: Option<usize>) -> Result<Statistics> {
     Ok(Statistics::new_unknown(&self.schema()))
 }
-````
+```
 
 **After:**
 
@@ -122,3 +123,36 @@ stats.column_statistics[0].min_value = ...;
 let mut stats = Arc::unwrap_or_clone(plan.partition_statistics(None)?);
 stats.column_statistics[0].min_value = ...;
 ```
+
+### Avro API and timestamp decoding changes
+
+DataFusion has switched to use `arrow-avro` (see [#17861]) when reading Avro
+files which results in a few changes:
+
+- `DataFusionError::AvroError` has been removed.
+- `From<apache_avro::Error> for DataFusionError` has been removed.
+- Avro crate re-export changed:
+  - Before: `datafusion::apache_avro`
+  - After: `datafusion::arrow_avro`
+- **Avro timestamp logical type interpretation changed.** Notable effects:
+  - Avro `timestamp-*` logical types are read as UTC timezone-aware Arrow
+    timestamps (`Timestamp(..., Some("+00:00"))`)
+  - Avro `local-timestamp-*` logical types remain timezone-naive
+    (`Timestamp(..., None)`)
+
+**Who is affected:**
+
+- Users matching on `DataFusionError::AvroError`
+- Users importing `datafusion::apache_avro`
+- Users relying on previous Avro timestamp logical type behavior
+
+**Migration guide:**
+
+- Replace `datafusion::apache_avro` imports with `datafusion::arrow_avro`.
+- Update error handling code that matches on `DataFusionError::AvroError` to use
+  the current error surface.
+- Validate timestamp handling where timezone semantics matter:
+  `timestamp-*` is UTC timezone-aware, while `local-timestamp-*` is
+  timezone-naive.
+
+[#17861]: https://github.com/apache/datafusion/pull/17861