You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by HanumathRao <gi...@git.apache.org> on 2018/02/08 19:12:38 UTC

[GitHub] drill issue #1117: DRILL-6089 Removed ordering trait from HashJoin in planne...

Github user HanumathRao commented on the issue:

    https://github.com/apache/drill/pull/1117
  
    @ilooner Changes look fine to me. However, as discussed offline I couldn't reproduce the plan which was sorting one of the inputs for a HASHJOIN. This might be because in the hashjoinprule we are not asking for those traits in the first place. However, I do agree that this fix can be a safegaurd in case if that plan happens to show up in some scenarios.
    
    
    Thanks for the fix. 


---