LEXIO-38100 Don't deepcopy expression in alias method (#12)

* nocopy * Version bumped to 0.12.0 Co-authored-by: ns-circle-ci <[email protected]>
NarrativeScience-old · May 26, 2022 · 78f2c84 · 78f2c84
1 parent e7ad07e
commit 78f2c84
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 9 deletions.
diff --git a/pyproject.toml b/pyproject.toml
@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "pysaql"
-version = "0.11.0"
+version = "0.12.0"
 description = "Python SAQL query builder"
 authors = ["Jonathan Drake <[email protected]>"]
 license = "BSD-3-Clause"

diff --git a/pysaql/__init__.py b/pysaql/__init__.py
@@ -1,3 +1,3 @@
 """Python SAQL query builder"""
 
-__version__ = "0.11.0"
+__version__ = "0.12.0"
diff --git a/pysaql/expression.py b/pysaql/expression.py
@@ -2,7 +2,6 @@
 
 
 from abc import ABC, abstractmethod
-from copy import deepcopy
 from typing import Optional
 
 from .util import escape_identifier
@@ -20,19 +19,15 @@ class Expression(ABC):
     def alias(self, name: str) -> "Expression":
         """Set the alias name
 
-        This creates and returns a new expression object so a single field can be
-        aliased multiple times.
-
         Args:
             name: Alias name
 
         Returns:
             new expression object with alias
 
         """
-        new_expr = deepcopy(self)
-        new_expr._alias = name
-        return new_expr
+        self._alias = name
+        return self
 
     @abstractmethod
     def to_string(self) -> str: