Log HTTPException from OpenFile and Write in caching filesystem

Wrap the remote OpenFile call and both Write overloads with a try
block that emits a PGDUCK_SERVER_WARN for ExceptionType::HTTP and
rethrows. DuckDB's HTTPException message already carries URL, status
code, and reason, so a 404, 403, or 500 on an S3 object now surfaces
a single WARNING line before the error propagates to the client.
Non-HTTP exceptions pass through silently.

Signed-off-by: Marco Slot <marco.slot@snowflake.com>

Author: sfc-gh-mslot

duckdb_pglake/src/fs/caching_file_system.cpp

@@ -120,7 +120,17 @@ PGLakeCachingFileSystem::OpenFile(const string &fullUrl,
 	else
 	{
 		/* create a handle for the remote file */
-		wrappedHandle = remoteFs->OpenFile(url, openFlags, opener);
+		try
+		{
+			wrappedHandle = remoteFs->OpenFile(url, openFlags, opener);
+		}
+		catch (Exception &ex)
+		{
+			ErrorData error(ex);
+			if (error.Type() == ExceptionType::HTTP)
+				PGDUCK_SERVER_WARN("%.500s", error.Message().c_str());
+			throw;
+		}
 
 		if (requestCache)
 		{

duckdb_pglake/src/include/pg_lake/fs/caching_file_system.hpp

@@ -172,7 +172,17 @@ class PGLakeCachingFileSystem : public FileSystem {
 		if (pg_lakeHandle.context->interrupted)
 			throw InterruptException();
 
-		wrappedHandle.file_system.Write(wrappedHandle, buffer, byteCount, location);
+		try
+		{
+			wrappedHandle.file_system.Write(wrappedHandle, buffer, byteCount, location);
+		}
+		catch (Exception &ex)
+		{
+			ErrorData error(ex);
+			if (error.Type() == ExceptionType::HTTP)
+				PGDUCK_SERVER_WARN("%.500s", error.Message().c_str());
+			throw;
+		}
     }
 
 	int64_t Write(FileHandle &handle, void *buffer, int64_t byteCount) override {
@@ -220,7 +230,17 @@ class PGLakeCachingFileSystem : public FileSystem {
 			CleanUpCacheOnWriteFile(pg_lakeHandle);
 		}
 
-		return pg_lakeHandle.wrappedHandle->Write(buffer, byteCount);
+		try
+		{
+			return pg_lakeHandle.wrappedHandle->Write(buffer, byteCount);
+		}
+		catch (Exception &ex)
+		{
+			ErrorData error(ex);
+			if (error.Type() == ExceptionType::HTTP)
+				PGDUCK_SERVER_WARN("%.500s", error.Message().c_str());
+			throw;
+		}
     }
 
 	bool CanHandleFile(const string &fpath) override {