nix attr accessor for now

Author: paleolimbot

python/sedonadb/python/sedonadb/expr/expression.py

@@ -329,9 +329,11 @@ def __len__(self) -> int:
         )
 
     # Nested expressions
-    def __getitem__(self, key: Union[int, str, "Expr"]) -> "Expr":
+    def __getitem__(self, key: Union[int, str]) -> "Expr":
         if isinstance(key, int):
             # Python uses 0-based indexing; SQL uses 1-based indexing
+            if key < 0:
+                raise ValueError("Can't index array expression with negative integer")
             return self.funcs.array_extract(key + 1)
         elif isinstance(key, str):
             # get_field works for both structs and maps, returning a scalar
@@ -342,9 +344,6 @@ def __getitem__(self, key: Union[int, str, "Expr"]) -> "Expr":
                 "or .funcs.get_field() to extract with an expression key."
             )
 
-    def __getattr__(self, name: str) -> "Expr":
-        return self.funcs.get_field(name)
-
 
 class SortExpr:
     """A sort key — an `Expr` plus direction and null placement.

python/sedonadb/tests/expr/test_expression.py

@@ -366,7 +366,7 @@ def test_nested_expression_exec(con):
     )
 
     pd.testing.assert_frame_equal(
-        t.select(a=t.struct_col.a).to_pandas(), pd.DataFrame({"a": [1, 3]})
+        t.select(a=t.struct_col["a"]).to_pandas(), pd.DataFrame({"a": [1, 3]})
     )
 
     pd.testing.assert_frame_equal(