Skip to content

Commit

Permalink
fixed req-in-mem-distinct logic
Browse files Browse the repository at this point in the history
  • Loading branch information
Druue committed Dec 1, 2023
1 parent 439eeb0 commit 14c206b
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions query-engine/query-structure/src/query_arguments.rs
Original file line number Diff line number Diff line change
Expand Up @@ -76,8 +76,7 @@ impl QueryArguments {
}

pub fn requires_inmemory_distinct(&self) -> bool {
self.has_distinct()
&& ((!self.has_distinct_capability() && !self.has_distincton_preview()) || self.has_orderby())
!self.has_distincton_preview() || !self.has_distinct_capability() || self.has_orderby()
}

fn has_distincton_preview(&self) -> bool {
Expand All @@ -97,10 +96,6 @@ impl QueryArguments {
.has_capability(ConnectorCapability::DistinctOn)
}

fn has_distinct(&self) -> bool {
self.distinct.is_some()
}

fn has_orderby(&self) -> bool {
!self.order_by.is_empty()
}
Expand Down

0 comments on commit 14c206b

Please sign in to comment.