Fix dot-to-slash conversion to only apply to dataset references, not file paths

Co-authored-by: joocer <[email protected]>

Author: Copilot

opteryx/connectors/aws_s3_connector.py

@@ -86,7 +86,12 @@ def __init__(self, credentials=None, **kwargs):
             )
 
         self.minio = Minio(end_point, access_key, secret_key, secure=secure)
-        self.dataset = self.dataset.replace(".", OS_SEP)
+        
+        # Only convert dots to path separators if the dataset doesn't already contain slashes
+        # Dataset references like "my.dataset.table" use dots as separators
+        # File paths like "bucket/path/file.parquet" already have slashes and should not be converted
+        if OS_SEP not in self.dataset and "/" not in self.dataset:
+            self.dataset = self.dataset.replace(".", OS_SEP)
 
         # Check if dataset contains wildcards
         self.has_wildcards = paths.has_wildcards(self.dataset)

opteryx/connectors/gcp_cloudstorage_connector.py

@@ -91,7 +91,11 @@ def __init__(self, credentials=None, **kwargs):
         Asynchronous.__init__(self, **kwargs)
         Statistics.__init__(self, **kwargs)
 
-        self.dataset = self.dataset.replace(".", OS_SEP)
+        # Only convert dots to path separators if the dataset doesn't already contain slashes
+        # Dataset references like "my.dataset.table" use dots as separators
+        # File paths like "bucket/path/file.parquet" already have slashes and should not be converted
+        if OS_SEP not in self.dataset and "/" not in self.dataset:
+            self.dataset = self.dataset.replace(".", OS_SEP)
         self.credentials = credentials
 
         # Check if dataset contains wildcards