fix: telemetry no longer blocks extension load

std::async returns a future whose destructor blocks until the task
completes, so binding it to a local made the telemetry POST effectively
synchronous: extension loading waited for the HTTP request. With the
previous defaults (30s timeout, 3 retries, 4x backoff) the worst case
was roughly two minutes of blocked LOAD.

Use a detached std::thread instead, and bound the request to a 10s
timeout with retries disabled, since telemetry is best-effort.

Bump version to 2026072501.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: rustyconover

src/airport_extension.cpp

@@ -17,7 +17,7 @@
 #include "airport_logging.hpp"
 #include "query_farm_telemetry.hpp"
 
-#define AIRPORT_EXTENSION_VERSION "2026042701"
+#define AIRPORT_EXTENSION_VERSION "2026072501"
 
 namespace duckdb
 {

src/query_farm_telemetry.cpp

@@ -6,7 +6,6 @@
 #include "duckdb/main/extension_helper.hpp"
 #include "duckdb/main/config.hpp"
 #include <cstdlib>
-#include <future>
 using namespace duckdb_yyjson; // NOLINT
 
 namespace duckdb
@@ -25,6 +24,10 @@ namespace duckdb
 
 			auto &http_util = HTTPUtil::Get(*db);
 			unique_ptr<HTTPParams> params = http_util.InitializeParameters(*db, TARGET_URL);
+			// Telemetry is best-effort: bound the request so a stalled connection cannot
+			// keep the detached thread alive. No retries -- a dropped ping is not worth one.
+			params->timeout = 10;
+			params->retries = 0;
 
 			PostRequestInfo post_request(TARGET_URL, headers, *params, reinterpret_cast<const_data_ptr_t>(json_body),
 																	 json_body_size);
@@ -94,10 +97,12 @@ namespace duckdb
 		yyjson_mut_doc_free(doc);
 
 #ifndef __EMSCRIPTEN__
-		[[maybe_unused]] auto _ = std::async(
-				std::launch::async, [db_ptr = loader.GetDatabaseInstance().shared_from_this(), json = telemetry_data,
-														 len = telemetry_len]() mutable
-				{ sendHTTPRequest(std::move(db_ptr), json, len); });
+		// Fire-and-forget. Deliberately not std::async: the future it returns has a
+		// blocking destructor, which would make the caller wait on this HTTP request.
+		std::thread([db_ptr = loader.GetDatabaseInstance().shared_from_this(), json = telemetry_data,
+					len = telemetry_len]() mutable
+					{ sendHTTPRequest(std::move(db_ptr), json, len); })
+				.detach();
 #else
 		sendHTTPRequest(loader.GetDatabaseInstance().shared_from_this(), telemetry_data, telemetry_len);
 #endif