Fix or remove estimateCardinality(vector<MatchExpression*>)

    • Type: Improvement
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Query Optimization
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Method 
      SamplingEstimatorImpl::estimateCardinality(const std::vector<const MatchExpression*>& expressions)
      incorrectly stacks all estimates of all expressions onto the CE stack.
      They are completely disassociated from the QSNs where they were defined.
       
      In addition, the performance benefit from this method is not clear, and it requires collecting all MEs in advance. Not clear if this complexity and overhead would pay off.

            Assignee:
            Unassigned
            Reporter:
            Timour Katchaounov
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: