fix(vortex-geo): cache MultiPolygon extension id via CachedId

`ExtVTable::id` interned the id with `ExtId::new_static` on every call,
tripping the `disallowed_methods` clippy lint added on the base branch
(#8617) since it grabs the interner lock per call. Match the `Point` and
`Polygon` types by caching the id in a `static CachedId`.

Signed-off-by: Nemo Yu <zyu379@wisc.edu>

Author: HarukiMoriarty

vortex-geo/src/extension/multipolygon.rs

@@ -67,7 +67,8 @@ impl ExtVTable for MultiPolygon {
     type NativeValue<'a> = &'a ScalarValue;
 
     fn id(&self) -> ExtId {
-        ExtId::new_static("vortex.geo.multipolygon")
+        static ID: CachedId = CachedId::new("vortex.geo.multipolygon");
+        *ID
     }
 
     fn serialize_metadata(&self, metadata: &Self::Metadata) -> VortexResult<Vec<u8>> {
@@ -238,7 +239,6 @@ impl ArrowExportVTable for MultiPolygon {
             .arrow()
             .execute_arrow(storage, Some(&storage_field), ctx)?;
 
-        // Round-trip through GeoArrow's multipolygon array; `into_arrow` is concrete, so wrap in `Arc`.
         let multipolygons =
             MultiPolygonArray::try_from((arrow_storage.as_ref(), multipolygon_meta))
                 .map_err(|e| vortex_err!("failed to construct MultiPolygonArray: {e}"))?;