Skip to content

Commit 1b18573

Browse files
committed
fix: remove unnecessary serializer
1 parent 2a55dfb commit 1b18573

File tree

1 file changed

+1
-17
lines changed

1 file changed

+1
-17
lines changed

pyiceberg/expressions/__init__.py

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
)
3434
from typing import Literal as TypingLiteral
3535

36-
from pydantic import ConfigDict, Field, field_serializer
36+
from pydantic import ConfigDict, Field
3737

3838
from pyiceberg.expressions.literals import (
3939
AboveMax,
@@ -328,22 +328,6 @@ def __new__(cls, left: BooleanExpression, right: BooleanExpression, *rest: Boole
328328
obj = super().__new__(cls)
329329
return obj
330330

331-
@field_serializer("left")
332-
def ser_left(self, left: BooleanExpression) -> Any:
333-
if isinstance(left, IcebergRootModel):
334-
return left.root
335-
if isinstance(left, IcebergBaseModel):
336-
return left.model_dump()
337-
return str(left)
338-
339-
@field_serializer("right")
340-
def ser_right(self, right: BooleanExpression) -> Any:
341-
if isinstance(right, IcebergRootModel):
342-
return right.root
343-
if isinstance(right, IcebergBaseModel):
344-
return right.model_dump()
345-
return str(right)
346-
347331
def __str__(self) -> str:
348332
"""Return the string representation of the Or class."""
349333
return f"{str(self.__class__.__name__)}(left={repr(self.left)}, right={repr(self.right)})"

0 commit comments

Comments
 (0)