perf: cache static identifiers to avoid per-call interner locks

`id()` methods interned their `Id` via `Id::new`/`Id::new_static` on every
call, locking the global interner on hot paths. Use a `CachedId` static
that interns once and returns the cached `Id` lock-free; same value.

Refs: #8380

Signed-off-by: Han Damin <miniex@daminstudio.net>

Author: miniex

vortex-array/src/aggregate_fn/fns/all_nan/mod.rs

@@ -3,6 +3,7 @@
 
 use vortex_error::VortexResult;
 use vortex_session::VortexSession;
+use vortex_session::registry::CachedId;
 
 use crate::ArrayRef;
 use crate::Columnar;
@@ -34,7 +35,8 @@ impl AggregateFnVTable for AllNan {
     type Partial = bool;
 
     fn id(&self) -> AggregateFnId {
-        AggregateFnId::new("vortex.all_nan")
+        static ID: CachedId = CachedId::new("vortex.all_nan");
+        *ID
     }
 
     fn serialize(&self, _options: &Self::Options) -> VortexResult<Option<Vec<u8>>> {

vortex-array/src/aggregate_fn/fns/all_non_distinct/mod.rs

@@ -19,6 +19,7 @@ use std::sync::LazyLock;
 use vortex_error::VortexResult;
 use vortex_error::vortex_bail;
 use vortex_error::vortex_err;
+use vortex_session::registry::CachedId;
 
 use self::bool::check_bool_identical;
 use self::decimal::check_decimal_identical;
@@ -121,7 +122,8 @@ impl AggregateFnVTable for AllNonDistinct {
     type Partial = AllNonDistinctPartial;
 
     fn id(&self) -> AggregateFnId {
-        AggregateFnId::new("vortex.all_non_distinct")
+        static ID: CachedId = CachedId::new("vortex.all_non_distinct");
+        *ID
     }
 
     fn serialize(&self, _options: &Self::Options) -> VortexResult<Option<Vec<u8>>> {

vortex-array/src/aggregate_fn/fns/all_non_nan/mod.rs

@@ -3,6 +3,7 @@
 
 use vortex_error::VortexResult;
 use vortex_session::VortexSession;
+use vortex_session::registry::CachedId;
 
 use crate::ArrayRef;
 use crate::Columnar;
@@ -34,7 +35,8 @@ impl AggregateFnVTable for AllNonNan {
     type Partial = bool;
 
     fn id(&self) -> AggregateFnId {
-        AggregateFnId::new("vortex.all_non_nan")
+        static ID: CachedId = CachedId::new("vortex.all_non_nan");
+        *ID
     }
 
     fn serialize(&self, _options: &Self::Options) -> VortexResult<Option<Vec<u8>>> {

vortex-array/src/aggregate_fn/fns/all_non_null/mod.rs

@@ -3,6 +3,7 @@
 
 use vortex_error::VortexResult;
 use vortex_session::VortexSession;
+use vortex_session::registry::CachedId;
 
 use crate::ArrayRef;
 use crate::Columnar;
@@ -26,7 +27,8 @@ impl AggregateFnVTable for AllNonNull {
     type Partial = bool;
 
     fn id(&self) -> AggregateFnId {
-        AggregateFnId::new("vortex.all_non_null")
+        static ID: CachedId = CachedId::new("vortex.all_non_null");
+        *ID
     }
 
     fn serialize(&self, _options: &Self::Options) -> VortexResult<Option<Vec<u8>>> {

vortex-array/src/aggregate_fn/fns/all_null/mod.rs

@@ -3,6 +3,7 @@
 
 use vortex_error::VortexResult;
 use vortex_session::VortexSession;
+use vortex_session::registry::CachedId;
 
 use crate::ArrayRef;
 use crate::Columnar;
@@ -26,7 +27,8 @@ impl AggregateFnVTable for AllNull {
     type Partial = bool;
 
     fn id(&self) -> AggregateFnId {
-        AggregateFnId::new("vortex.all_null")
+        static ID: CachedId = CachedId::new("vortex.all_null");
+        *ID
     }
 
     fn serialize(&self, _options: &Self::Options) -> VortexResult<Option<Vec<u8>>> {

vortex-array/src/aggregate_fn/fns/bounded_max/mod.rs

@@ -13,6 +13,7 @@ use vortex_error::VortexResult;
 use vortex_error::vortex_bail;
 use vortex_error::vortex_ensure;
 use vortex_session::VortexSession;
+use vortex_session::registry::CachedId;
 
 use crate::ArrayRef;
 use crate::Columnar;
@@ -125,7 +126,8 @@ impl AggregateFnVTable for BoundedMax {
     type Partial = BoundedMaxPartial;
 
     fn id(&self) -> AggregateFnId {
-        AggregateFnId::new("vortex.bounded_max")
+        static ID: CachedId = CachedId::new("vortex.bounded_max");
+        *ID
     }
 
     fn serialize(&self, options: &Self::Options) -> VortexResult<Option<Vec<u8>>> {

vortex-array/src/aggregate_fn/fns/bounded_min/mod.rs

@@ -11,6 +11,7 @@ use vortex_error::VortexExpect;
 use vortex_error::VortexResult;
 use vortex_error::vortex_ensure;
 use vortex_session::VortexSession;
+use vortex_session::registry::CachedId;
 
 use crate::ArrayRef;
 use crate::Columnar;
@@ -79,7 +80,8 @@ impl AggregateFnVTable for BoundedMin {
     type Partial = BoundedMinPartial;
 
     fn id(&self) -> AggregateFnId {
-        AggregateFnId::new("vortex.bounded_min")
+        static ID: CachedId = CachedId::new("vortex.bounded_min");
+        *ID
     }
 
     fn serialize(&self, options: &Self::Options) -> VortexResult<Option<Vec<u8>>> {

vortex-array/src/aggregate_fn/fns/count/mod.rs

@@ -5,6 +5,7 @@ mod grouped;
 pub(crate) use grouped::CountGroupedKernel;
 use vortex_error::VortexExpect;
 use vortex_error::VortexResult;
+use vortex_session::registry::CachedId;
 
 use crate::ArrayRef;
 use crate::Columnar;
@@ -41,7 +42,8 @@ impl AggregateFnVTable for Count {
     type Partial = CountPartial;
 
     fn id(&self) -> AggregateFnId {
-        AggregateFnId::new("vortex.count")
+        static ID: CachedId = CachedId::new("vortex.count");
+        *ID
     }
 
     fn serialize(&self, _options: &Self::Options) -> VortexResult<Option<Vec<u8>>> {

vortex-array/src/aggregate_fn/fns/first/mod.rs

@@ -2,6 +2,7 @@
 // SPDX-FileCopyrightText: Copyright the Vortex contributors
 
 use vortex_error::VortexResult;
+use vortex_session::registry::CachedId;
 
 use crate::ArrayRef;
 use crate::Columnar;
@@ -40,7 +41,8 @@ impl AggregateFnVTable for First {
     type Partial = FirstPartial;
 
     fn id(&self) -> AggregateFnId {
-        AggregateFnId::new("vortex.first")
+        static ID: CachedId = CachedId::new("vortex.first");
+        *ID
     }
 
     fn serialize(&self, _options: &Self::Options) -> VortexResult<Option<Vec<u8>>> {

vortex-array/src/aggregate_fn/fns/is_constant/mod.rs

@@ -13,6 +13,7 @@ mod varbin;
 use vortex_error::VortexExpect;
 use vortex_error::VortexResult;
 use vortex_error::vortex_bail;
+use vortex_session::registry::CachedId;
 
 use self::bool::check_bool_constant;
 use self::decimal::check_decimal_constant;
@@ -258,7 +259,8 @@ impl AggregateFnVTable for IsConstant {
     type Partial = IsConstantPartial;
 
     fn id(&self) -> AggregateFnId {
-        AggregateFnId::new("vortex.is_constant")
+        static ID: CachedId = CachedId::new("vortex.is_constant");
+        *ID
     }
 
     fn serialize(&self, _options: &Self::Options) -> VortexResult<Option<Vec<u8>>> {