You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
accept aggregation in ConnectorMetadata#applyAggregation
However, for some connectors this will be hard to achieve.
For example, for a database doing case insensitive comparisons, we may need to
retain Filter on the Presto side.
In such a case, we could attempt ConnectorMetadata#applyAggregation "through Filter"
leveraging the fact that the filter is on the aggregation keys.
For example, for a database doing case insensitive comparisons, we may need to
retain Filter on the Presto side.
In such a case, we could attempt ConnectorMetadata#applyAggregation "through Filter"
leveraging the fact that the filter is on the aggregation keys.
In such a case, pushing down aggregation would not be correct.
Consider query
which will be planned as:
The ideal connector behavior will be to
ConnectorMetadata#applyFilter
ConnectorMetadata#applyAggregation
However, for some connectors this will be hard to achieve.
For example, for a database doing case insensitive comparisons, we may need to
retain Filter on the Presto side.
In such a case, we could attempt
ConnectorMetadata#applyAggregation
"through Filter"leveraging the fact that the filter is on the aggregation keys.
(Thanks @sopel39 for spotting this!)
Relates to: #4137
The text was updated successfully, but these errors were encountered: