spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sathiyapk <>
Subject [GitHub] spark issue #19451: SPARK-22181 Adds ReplaceExceptWithNotFilter rule
Date Tue, 10 Oct 2017 21:42:13 GMT
Github user sathiyapk commented on the issue:
    @gengliangwang thanks for your review. I pushed a new commit, please let me know if it
addresses your comments. And regarding
    > Also please use semanticEquals instead of sameResult
    I couldn't call `semanticEquals ` directly on a LogicalPlan, can you please give an example
how to use it efficiently?
    > I think CombineFilters(plan) can finish in one pass.
    CombineFilters(plan) combines two neighbouring filters into one, so assuming there can
be any number of filters, we would need a recursive call, so more than one pass. Correct me
if i'm wrong, thanks.


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message