docs: add clarification around SQL timestamps (#1012)

googleapis · Aug 8, 2024 · 6e80190 · 6e80190
1 parent 45bc8c4
commit 6e80190
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/google/cloud/bigtable/data/execute_query/metadata.py b/google/cloud/bigtable/data/execute_query/metadata.py
@@ -214,6 +214,12 @@ class Bool(Type):
         type_field_name = "bool_type"
 
     class Timestamp(Type):
+        """
+        Timestamp supports :class:`DatetimeWithNanoseconds` but Bigtable SQL does
+        not currently support nanoseconds precision. We support this for potential
+        compatibility in the future. Nanoseconds are currently ignored.
+        """
+
         type_field_name = "timestamp_type"
         expected_types = (
             datetime.datetime,

diff --git a/google/cloud/bigtable/data/execute_query/values.py b/google/cloud/bigtable/data/execute_query/values.py
@@ -100,6 +100,9 @@ def __repr__(self) -> str:
     bool,
     bytes,
     str,
+    # Note that Bigtable SQL does not currently support nanosecond precision,
+    # only microseconds. We use this for compatibility with potential future
+    # support
     DatetimeWithNanoseconds,
     date_pb2.Date,
     "Struct",