From df66e92ca2c334b591fb6f992bfbe16b6f338c95 Mon Sep 17 00:00:00 2001 From: Jim Bosch Date: Thu, 4 Apr 2024 16:00:09 -0400 Subject: [PATCH] Defer guards on QueryBuilder distinct vs. group_by to make them useful. --- python/lsst/daf/butler/direct_query_driver/_query_builder.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/python/lsst/daf/butler/direct_query_driver/_query_builder.py b/python/lsst/daf/butler/direct_query_driver/_query_builder.py index 3f17f707cd..68ae35a50c 100644 --- a/python/lsst/daf/butler/direct_query_driver/_query_builder.py +++ b/python/lsst/daf/butler/direct_query_driver/_query_builder.py @@ -104,9 +104,6 @@ class QueryBuilder: queries that have no real columns. """ - def __post_init__(self) -> None: - assert not (self.distinct and self.group_by), "At most one of distinct and group_by can be set." - @classmethod def handle_empty_columns( cls, columns: list[sqlalchemy.sql.ColumnElement] @@ -139,6 +136,7 @@ def select(self) -> sqlalchemy.Select: select : `sqlalchemy.Select` SQLAlchemy SELECT statement. """ + assert not (self.distinct and self.group_by), "At most one of distinct and group_by can be set." if self.joiner.name_shrinker is None: self.joiner.name_shrinker = self.joiner._make_name_shrinker() sql_columns: list[sqlalchemy.ColumnElement[Any]] = []