From 0ea932dafdac1ba3f86135cded890012d9733fdc Mon Sep 17 00:00:00 2001
From: Shiduo Li <sli999@gatech.edu>
Date: Fri, 23 Oct 2026 23:47:41 -0700
Subject: [PATCH] fix(rust): compile jobs without remote feature (#4103)

## What

Move typed job decode error construction behind a feature-aware helper
so builds without the `remote` feature do not reference `Error::Http`.
Local decode failures continue to surface as `Error::Runtime`, while
remote builds retain HTTP errors with request IDs.

Add a regression test for local typed-result decode failures.

Closes #4096.

## Testing

- `cargo fmt --all -- --check`
- `cargo check -p lancedb --no-default-features`
- `cargo test -p lancedb --no-default-features
job::tests::local_typed_result_decode_errors_are_runtime_errors`

Co-authored-by: Xuanwo <github@xuanwo.io>
---
 rust/lancedb/src/job.rs | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/rust/lancedb/src/job.rs b/rust/lancedb/src/job.rs
index 6865e59ee..d671365f9 100644
--- a/rust/lancedb/src/job.rs
+++ b/rust/lancedb/src/job.rs
@@ -636,6 +636,14 @@ mod tests {
 
     use super::*;
 
+    #[test]
+    fn local_typed_result_decode_errors_are_runtime_errors() {
+        assert!(matches!(
+            TerminalResult::local(Value::Null).decode::<u64>(),
+            Err(Error::Runtime { .. })
+        ));
+    }
+
     #[tokio::test]
     async fn mapped_spawned_job_reuses_outcome() {
         let job = Job::spawned(tokio::spawn(async { Ok(41_u64) })).map(|value| value + 1);